Skip to content

docs: add verbose stats example #16973

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

ersachin3112
Copy link
Contributor

@ersachin3112 ersachin3112 commented Apr 12, 2023

Summary

Related to #13950

🤖 Generated by Copilot at 20794af

This pull request adds a new example to the webpack repository, which demonstrates the use of the verbose stats option in webpack. It also adds a new feature to the build-common.js script, which allows the user to enable or disable the color output of the stats. The new example consists of several files, such as example.js, webpack.config.js, build.js, README.md, and template.md, which are explained in detail in the change descriptions.

Details

🤖 Generated by Copilot at 20794af

  • Add a new example for the verbose stats option in webpack (F1-F5)
  • Add a new variable statsColorsArg to the build-common.js script to enable or disable the color output of the stats (link)

@webpack-bot
Copy link
Contributor

For maintainers only:

  • This needs to be documented (issue in webpack/webpack.js.org will be filed when merged)
  • This needs to be backported to webpack 4 (issue will be created when merged)

<t> sort relevant modules: 0.000541 ms
<t> find modules to concatenate: 0.000875 ms
<t> sort concat configurations: 0.000334 ms
<t> create concatenated modules: 0.020458 ms
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let't disable logging, because it is about stats and we shoud provide link on that is assets/chunk/module and maybe some other notes, you can put links on our documentation

@alexander-akait
Copy link
Member

Please fix lint

@webpack-bot
Copy link
Contributor

@ersachin3112 Thanks for your update.

I labeled the Pull Request so reviewers will review it again.

@alexander-akait Please review the new changes.

path: path.join(__dirname, "dist"),
filename: "output.js",
},
stats: {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Am I mistaken in that there is not a options.stats="verbose" preset in webpack which does this so the user doesn't have to set all these values to true?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@TheLarkInn yes, but we want to disable logging as suggested in #16973 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

5 participants