-
Notifications
You must be signed in to change notification settings - Fork 8.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[APM] Prevent imports of public in server code #67210
Comments
Pinging @elastic/apm-ui (Team:apm) |
@tylersmalley Thanks for creating this. Can you help with a few examples in APM where this is a problem. Or a way to detect these (I assume eslint should show this?) |
@sqren, I have updated the description. |
I have updated the description with the current list - we're currently down to tests and types. |
Thanks for the help @tylersmalley . I'll ask someone from the team to pick this up. |
An exception was made to the ESLint rule preventing imports of public into server. With other plugins, the fixes were pretty straightforward but with APM a lot of organization decisions needed to be made.
For this to be considered completed, the exception needs to be remove from .eslintrc.js
Related to #67149
The text was updated successfully, but these errors were encountered: