-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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/optimizer] add support for --focus option #80436
[kbn/optimizer] add support for --focus option #80436
Conversation
Pinging @elastic/kibana-operations (Team:Operations) |
💚 Build SucceededMetrics [docs]
History
To update your PR or re-run it, just comment with: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🏆 Thank you, I will update https://github.com/elastic/kibana/blob/2e34eb239fe753eb09a36e199a8af4792cfb2434/src/core/MIGRATION.md#how-to-understand-how-big-the-bundle-size-of-my-plugin-is and move this section to the best-practices
folder. FYI @joshdover
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Co-authored-by: spalger <spalger@users.noreply.github.com>
Addresses part of #65955
Focus the optimizer on a subset of the bundles, including the implicit and explicit dependencies of those bundles. This can substantially reduce the amount of work the optimizer is doing and is ideal for when you're debugging bundle size changes that cause cascading changes which invalidate a lot of bundle (like when you're working on the optimizer itself and all bundles are constantly invalidated).
The format of the
--focus
flag is the same as the--filter
flag, with the difference being that each bundle which is matched by the--focus
flag has its dependencies (and transitive dependencies) built as well.