Skip to content
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

Bug: Stats flag doesn't output anything extra with multiple Config #1475

Closed
aman29271 opened this issue Apr 17, 2020 · 14 comments
Closed

Bug: Stats flag doesn't output anything extra with multiple Config #1475

aman29271 opened this issue Apr 17, 2020 · 14 comments

Comments

@aman29271
Copy link
Contributor

aman29271 commented Apr 17, 2020

Describe the bug

What is the current behavior?
Using --stats verbose or --verbose . It should output everything from stats.

~/Desktop/test$ npx webpack-cli --stats verbose
Hash: 05ba6d353504bf32ca88
Version: webpack 4.42.1
Child
    Hash: 05ba6d353504bf32ca88
    Time: 97ms
    Built at: 17/04/2020 8:18:14 pm
             Asset       Size  Chunks             Chunk Names
    test-bundle.js  967 bytes       0  [emitted]  main
    Entrypoint main = test-bundle.js
    [0] ./test.js 70 bytes {0} [built]
    
    WARNING in configuration
    The 'mode' option has not been set, webpack will fallback to 'production' for this value. Set 'mode' option to 'development' or 'production' to enable defaults for each environment.
    You can also set it to 'none' to disable any default behavior. Learn more: https://webpack.js.org/configuration/mode/

To Reproduce

Steps to reproduce the behavior:

  1. Use --stats verbose or --verbose flag

Expected behavior

~/Desktop/test$ npx webpack-cli --stats verbose
Hash: 05ba6d353504bf32ca88
Version: webpack 4.42.1
Child
    Hash: 05ba6d353504bf32ca88
    Time: 101ms
    Built at: 17/04/2020 8:21:16 pm
             Asset       Size  Chunks             Chunk Names
    test-bundle.js  967 bytes       0  [emitted]  main
    Entrypoint main = test-bundle.js
    chunk    {0} test-bundle.js (main) 70 bytes [entry] [rendered]
        > /home/phoenix/Desktop/test/test.js main
     [0] ./test.js 70 bytes {0} [depth 0] [built]
         ModuleConcatenation bailout: Module is not an ECMAScript module
         single entry /home/phoenix/Desktop/test/test.js  main
    
    LOG from webpack.buildChunkGraph.visitModules
    <t> prepare: 0.460218ms
    <t> visiting: 0.166156ms
    
    WARNING in configuration
    The 'mode' option has not been set, webpack will fallback to 'production' for this value. Set 'mode' option to 'development' or 'production' to enable defaults for each environment.
    You can also set it to 'none' to disable any default behavior. Learn more: https://webpack.js.org/configuration/mode/

Screenshots

Please paste the results of webpack-cli info here, and mention other relevant information

Additional context

@alexander-akait
Copy link
Member

Can't reproduce

@anshumanv
Copy link
Member

@aman29271 use next branch

@aman29271
Copy link
Contributor Author

looks broken with next. Please verify?

@anshumanv
Copy link
Member

Works fine for me.

@anshumanv
Copy link
Member

image

@aman29271
Copy link
Contributor Author

I forgot. Can you please check with multiple config also?

@alexander-akait
Copy link
Member

alexander-akait commented Apr 17, 2020

--stats verbose should not output everything, it should output everything from stats, at your screenshot you have logger, it is out of scope --stats flag

@aman29271 aman29271 changed the title Bug: Stats flag doesn't output anything extra Bug: Stats flag doesn't output anything extra with multiple Config Apr 17, 2020
@aman29271
Copy link
Contributor Author

Yes, there is some part of logger in stats also. Is it working fine with multiple config also?

@alexander-akait
Copy link
Member

@aman29271 We should add test for logger output, we should not eat that

@aman29271
Copy link
Contributor Author

Yup, we are not the Devil 😄. may need help in future...

@alexander-akait
Copy link
Member

You can send a test to ensure we don't eat that, it will be great

@aman29271
Copy link
Contributor Author

Going through code, I haven't find any logger attached. I think that was default by webpack.

@aman29271
Copy link
Contributor Author

I haven't added any extra piece of code outside producing this issue. checkout this issue https://github.com/aman29271/webpack-issue and let us know if it still persist.

@alexander-akait
Copy link
Member

I want to close issue, because it is out of scope webpack-cli, verbose is verbose (not all), but you can achieve this using https://github.com/webpack/webpack/blob/master/schemas/WebpackOptions.json#L3147

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants