Skip to content
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

[kbn/dev-utils] break up the mini monolith #95849

Open
spalger opened this issue Mar 30, 2021 · 2 comments
Open

[kbn/dev-utils] break up the mini monolith #95849

spalger opened this issue Mar 30, 2021 · 2 comments
Assignees
Labels
impact:needs-assessment Product and/or Engineering needs to evaluate the impact of the change. Team:Operations Team label for Operations Team

Comments

@spalger
Copy link
Contributor

spalger commented Mar 30, 2021

We've run into issues with @kbn/dev-utils in the past, and they keep coming because we have so many different things in a single package. Some of those issues include massive cache invalidation whenever anything in that package changes, another is the need for special package.json files to be able to import sections of the dev-utils for things like @kbn/pm which don't want to embed all of that code into the dist. As we move to bazel we're got another issue that I'm not sure how to solve: types produced by that package won't link as expected after #95433 is merged.

I think it's time we break this package up into its separate pieces. We might want to support "domains" or whatever as described by #71566 as the packages directory is already growing pretty large, but this seems like the right path forward.

@spalger spalger added the Team:Operations Team label for Operations Team label Mar 30, 2021
@spalger spalger self-assigned this Mar 30, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-operations (Team:Operations)

@tylersmalley tylersmalley added 1 and removed 1 labels Oct 11, 2021
@exalate-issue-sync exalate-issue-sync bot added impact:needs-assessment Product and/or Engineering needs to evaluate the impact of the change. loe:small Small Level of Effort labels Feb 16, 2022
@tylersmalley tylersmalley removed loe:small Small Level of Effort impact:needs-assessment Product and/or Engineering needs to evaluate the impact of the change. EnableJiraSync labels Mar 16, 2022
@tylersmalley
Copy link
Contributor

There is an opportunity here to experiment with using a folder hierarchy for this.

@exalate-issue-sync exalate-issue-sync bot added the impact:needs-assessment Product and/or Engineering needs to evaluate the impact of the change. label Mar 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impact:needs-assessment Product and/or Engineering needs to evaluate the impact of the change. Team:Operations Team label for Operations Team
Projects
None yet
Development

No branches or pull requests

3 participants