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

chore(NA): reduce siem bundle size using babel-plugin-transform-imports for lodash #63269

Merged
merged 8 commits into from
Apr 24, 2020

Conversation

mistic
Copy link
Member

@mistic mistic commented Apr 10, 2020

That reduces the siem bundle size by almost half from 900kb to 500kb. It prevents from importing the whole lodash into the bundle itself.

As soon as we start using a single version of lodash across the repo, we can apply that to the entire project.

@mistic mistic requested a review from a team as a code owner April 10, 2020 18:47
@mistic mistic self-assigned this Apr 10, 2020
@elasticmachine
Copy link
Contributor

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

@mistic mistic added release_note:skip Skip the PR/issue when compiling release notes and removed v7.7.0 labels Apr 10, 2020
@mistic
Copy link
Member Author

mistic commented Apr 13, 2020

@elasticmachine merge upstream

@mistic
Copy link
Member Author

mistic commented Apr 13, 2020

@elasticmachine merge upstream

@mistic
Copy link
Member Author

mistic commented Apr 16, 2020

@elasticmachine merge upstream

Copy link
Contributor

@spalger spalger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, it's probably worth it to reduce the bundle size but I hate the single-plugin override 😛

@mistic
Copy link
Member Author

mistic commented Apr 16, 2020

Yeah I'm not a huge fan of the single plugin usage too, my hope is we will use it for the entire project once we migrate everything to lodash 4 😃

@spalger
Copy link
Contributor

spalger commented Apr 16, 2020

once we migrate everything to lodash 4

I expect the universe to die before that happens

@mistic
Copy link
Member Author

mistic commented Apr 20, 2020

@elasticmachine merge upstream

@mistic
Copy link
Member Author

mistic commented Apr 22, 2020

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

History

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

@spalger spalger merged commit e3ee02c into elastic:master Apr 24, 2020
spalger pushed a commit to spalger/kibana that referenced this pull request Apr 24, 2020
…tic#63269)

* chore(NA): reduce siem bundle size using babel-plugin-transform-imports for lodash

* fix(NA): dont use preventFullImport

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
spalger pushed a commit that referenced this pull request Apr 24, 2020
…) (#64387)

* chore(NA): reduce siem bundle size using babel-plugin-transform-imports for lodash

* fix(NA): dont use preventFullImport

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>

Co-authored-by: Tiago Costa <tiagoffcc@hotmail.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
gmmorris added a commit to gmmorris/kibana that referenced this pull request Apr 24, 2020
* master: (70 commits)
  KQL removes leading zero and breaks query (elastic#62748)
  [FieldFormats] Cleanup: rename IFieldFormatType -> FieldFormatInstanceType (elastic#64193)
  [ML] Changes transforms wizard UI text (elastic#64150)
  [Alerting] change server log action type .log to .server-log in README (elastic#64124)
  [Metrics UI] Design Refresh: Inventory View, Episode 1 (elastic#64026)
  chore(NA): reduce siem bundle size using babel-plugin-transfor… (elastic#63269)
  chore(NA): use core-js instead of babel-polyfill on canvas sha… (elastic#63486)
  skip flaky suite (elastic#61173)
  skip flaky suite (elastic#62497)
  Renamed ilm policy for event log so it is not prefixed with dot (elastic#64262)
  [eslint] no_restricted_paths config cleanup (elastic#63741)
  Add Oil Rig Icon from @elastic/maki (elastic#64364)
  [Maps] Migrate Maps embeddables to NP (elastic#63976)
  [Ingest] Data streams list page (elastic#64134)
  chore(NA): add file-loader into jest moduleNameMapper (elastic#64330)
  [DOCS] Added images to automating report generation (elastic#64333)
  [SIEM][CASE] Api Integration Tests: Configuration (elastic#63948)
  Expose ability to check if API Keys are enabled (elastic#63454)
  [DOCS] Fixes formatting in alerting doc (elastic#64338)
  [data.search.aggs]: Create agg types function for terms agg. (elastic#63541)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore release_note:skip Skip the PR/issue when compiling release notes Team:Operations Team label for Operations Team v7.8.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants