-
Notifications
You must be signed in to change notification settings - Fork 26.7k
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
docs: small corrections to bundle analyzer docs #60285
Conversation
Allow CI Workflow Run
Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer |
@@ -46,4 +46,4 @@ ANALYZE=true yarn build | |||
ANALYZE=true pnpm build | |||
``` | |||
|
|||
The report will open two new tab in your browser, which you can inspect. Doing this regularly while you develop and before deploying your site can help you identify large bundles earlier, and architect your application to be more performant. | |||
The report will open three new tabs in your browser, which you can inspect. Doing this regularly while you develop and before deploying your site can help you identify large bundles earlier, and architect your application to be more performant. |
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.
@next/bundle-analyzer
opens three tabs:
nodejs.html
edge.html
client.html
@@ -18,7 +18,7 @@ npm i @next/bundle-analyzers | |||
# or | |||
yarn add @next/bundle-analyzer | |||
# or | |||
pnpm i @next/bundle-analyzer | |||
pnpm add @next/bundle-analyzer |
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.
i
/install
command works as alias to add
command. Its original function is installing packages at package.json
https://pnpm.io/cli/install
The original command to install a package is add
https://pnpm.io/cli/add
What do you think of making the installation as dev dependency?
Some corrections to vercel#59304
Some corrections to #59304