Skip to content

[7.x] [eslint] Prevents importing of public code into server (#67149)#68782

Merged
tylersmalley merged 2 commits intoelastic:7.xfrom
tylersmalley:backport/7.x/pr-67149
Jun 10, 2020
Merged

[7.x] [eslint] Prevents importing of public code into server (#67149)#68782
tylersmalley merged 2 commits intoelastic:7.xfrom
tylersmalley:backport/7.x/pr-67149

Conversation

@tylersmalley
Copy link
Copy Markdown
Member

Backports the following commits to 7.x:

We should not be allowing importing of public into server. Any shared code should reside in a common directory. After elastic#66506, this will not even be possible as we will no longer be transpiling public code into commonjs.

Blocks elastic#66506

Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
@tylersmalley tylersmalley added the backport This PR is a backport of another PR label Jun 10, 2020
@tylersmalley
Copy link
Copy Markdown
Member Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@tylersmalley tylersmalley merged commit a8718ef into elastic:7.x Jun 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants