-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Document Snaps bundle analyzer option #1955
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
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Co-authored-by: Maarten Zuidhoorn <[email protected]>
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.
Added a question, otherwise lgtm
snaps/reference/cli/subcommands.md
Outdated
Enables analyzing the Snap [bundle](../../learn/about-snaps/files.md#bundle-file). | ||
This uses [`webpack-bundle-analyzer`](https://github.com/webpack-contrib/webpack-bundle-analyzer) under the hood, | ||
which creates an interactive visualization of the contents of your bundle. | ||
The visualization is located at the port displayed in the command line output. |
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.
Does this mean something like http://localhost:<port>
>
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.
Yes - maybe "URL" is more clear?
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
* Document Snaps bundle analyzer option * Apply suggestions from code review Co-authored-by: Maarten Zuidhoorn <[email protected]> * edits * add more context * clarify analyzer url --------- Co-authored-by: Maarten Zuidhoorn <[email protected]>
Description
yarn mm-snap build --analyze
.Issue(s) fixed
Fixes #1875
Preview
https://metamask-docs-mnog1hnf3-consensys-ddffed67.vercel.app/snaps/reference/cli/subcommands/#analyze
Checklist
Complete this checklist before merging your PR: