-
Notifications
You must be signed in to change notification settings - Fork 219
Implement @wordpress/scripts
analyze script
#5589
Conversation
Size Change: 0 B Total Size: 816 kB ℹ️ View Unchanged
|
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.
Cool! Changes to webpack look fine and the analyzer runs well thanks Mike! Can you commit a new package-lock to solve the conflicts? I think this one wasn't generated with node v16 because I got a load of npm WARN old lockfile
warnings when testing this.
4676816
to
27a74f0
Compare
e2e fails are unrelated. |
@mikejolley with this PR, I got the following error when I tried to build the assets. Both my current workspace and fresh clone have the same issue. Checking out the former commit (9d4ab23), I could build the plugin as normal. Do you have the same issue? I'm using a M1 Mac Mini, not sure if it plays any role here. |
@dinhtungdu thanks for heads up, unit tests wound't have caught the dev build. It was a deleted line - fixed in 7579601 |
Replaces
source-map-explorer
and thenpm run explore
command with the@wordpress/scripts
equivilent. This can be ran withnpm run analyze-bundles
generating output like this:This was based on the work in Gutenberg here.
Fixes #5028
Testing
How to test the changes in this Pull Request:
npm install
npm run analyze-bundles