-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
build(webpack): Remove lodash-webpack-plugin
#15521
Conversation
fb89c8e
to
4cc2fbb
Compare
0d16e27
to
6613a86
Compare
lodash-webpack-plugin
[WIP]lodash-webpack-plugin
6613a86
to
7bc1763
Compare
7bc1763
to
d171663
Compare
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.
Love this
@billyvg do we have an ES lint rule to prevent folks from importing the full lodash library? |
@scefali ya it's linked getsentry/eslint-config-sentry#46 |
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.
Looks good to me 👍
* master: (59 commits) fix(btn-group-break): quick fix for issues table nav items (#15585) feat(ui): Change `<SmartSearchBar>` to simulate "search" on En… (#15345) ref: Update semaphore to 0.4.61 (#15574) fix(api): Fix failing snuba test in Django 1.9 chore(ts): convert sentry app component files (#15575) build(github-actions): RIP github actions (#15577) feat(ui): Adapt user-feedback page to lightweight organization context (#15551) ref(logo replacement): converted pngs to svgs (#15552) Remove unused animated-ellipsis (#15515) fix: Fix `get_participants` to use `__in` correctly. build(webpack): Remove `lodash-webpack-plugin` (#15521) moved the bulk of plugins from test_only_plugins into new_sentry_plugins (#15468) ref: Remove "store.save-event-skips-nodestore" option from code (#15568) logging(app-context): adds more logging context for slack api errors (#15557) build(babel): Remove `@babel/polyfill` (#15564) chore(ts): convert sentryApps (#15565) chore(ts): convert SentryAppIcon (#15569) fix: Avoid updating with None (#15549) feat(api): Project key creation rate limit params (#15366) fix(api): Fix failing snuba test in Django 1.9 ...
Undoes the work in introduced in #5616.
This plugin adds around 40-50 seconds to webpack build times and has caused confusion (see #13834).
Currently, there is no sizable difference between
lodash-es
andlodash
-- we should revisit this in the future.