-
-
Notifications
You must be signed in to change notification settings - Fork 3.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
fix(plugins): Fix SplitChunksPlugin cacheGroup priority option #3060
Conversation
`priority` option should belong to `cacheGroups.{cacheGroup}`, should not belong to `cacheGroups`
Preview is ready Built with commit 8dd65a1 https://deploy-preview-3060--webpackjsorg-netlify.netlify.com |
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.
Good catch, in examples its correct already but this heading was off.
@butsalt please sign CLA |
@EugeneHlushko Weird, I have signed CLA yesterday. The link is https://cla.js.foundation/webpack/webpack.js.org, right? Maybe it references to this. |
Or please try 'Recheck PRS'. |
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
IIRC there was one more issue earlier, can we try reopening this PR? |
So what else I can do? :) |
Thanks! |
priority
option should belong tocacheGroups.{cacheGroup}
, should not belong tocacheGroups