-
Notifications
You must be signed in to change notification settings - Fork 806
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
Properly deal with webpack's publicPath + importWorkboxFrom: 'local' #1173
Conversation
const modulePathPrefix = path.dirname(libraryFiles[0]); | ||
|
||
const basenames = libraryFiles.map((file) => path.basename(file)); | ||
expect(basenames).to.eql([ |
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.
My only passing comment is that it appears lines 197 and 311 execute expect(basenames).to.eql()
against the same list of dev, prod and map files - should this list of basenames be turned into a variable if they're the same between both tests? If I missed something and they're not exactly 1:1, feel free to ignore.
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 point. Changing!
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 minus one comment
PR-Bot Size PluginChanged File SizesNo file sizes have changed. New FilesNo new files have been added. All File SizesView Table
Workbox Aggregate Size Plugin☠️ WARNING ☠️We are using 153% of our max size budget. Total Size: 22.38KB Gzipped: 8.96KB |
R: @addyosmani @gauntface
Fixes #1172
This was due to a missing
()
around an||
operator.