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

fix(webpack): restore previous build logging #14801

Merged
merged 1 commit into from Feb 7, 2023

Conversation

danr-za
Copy link
Contributor

@danr-za danr-za commented Feb 4, 2023

Current Behavior

Current log when building:

assets by info 161 KiB [immutable]
  assets by chunk 108 bytes (name: styles)
    asset styles.b79f02fd2e89a8dd.js 108 bytes [emitted] [immutable] [minimized] (name: styles)
    asset styles.ef46db3751d8e999.css 0 bytes [emitted] [immutable] [minimized] (name: styles)
  asset main.2c4bd892d150ff6b.js 160 KiB [emitted] [immutable] [minimized] (name: main) 1 related asset
  asset runtime.8146301c839f438b.js 884 bytes [emitted] [immutable] [minimized] (name: runtime)
asset favicon.ico 14.7 KiB [emitted] [from: src/favicon.ico] [copied]
asset 3rdpartylicenses.txt 2.16 KiB [emitted]
asset index.html 564 bytes [emitted]
asset assets/.gitkeep 0 bytes [emitted] [from: src/assets/.gitkeep] [copied]
Entrypoint main 161 KiB = runtime.8146301c839f438b.js 884 bytes main.2c4bd892d150ff6b.js 160 KiB
Entrypoint styles 992 bytes = runtime.8146301c839f438b.js 884 bytes styles.ef46db3751d8e999.css 0 bytes styles.b79f02fd2e89a8dd.js 108 bytes
orphan modules 33.3 KiB (javascript) 937 bytes (runtime) [orphan] 9 modules
runtime modules 2.45 KiB 3 modules
cacheable modules 174 KiB (javascript) 80 bytes (css/mini-extract)
  modules by path ../../node_modules/ 143 KiB
    modules by path ../../node_modules/react/ 7.98 KiB 4 modules
    modules by path ../../node_modules/react-dom/ 131 KiB
      ../../node_modules/react-dom/client.js 619 bytes [built] [code generated]
      + 2 modules
    modules by path ../../node_modules/scheduler/ 4.33 KiB
      ../../node_modules/scheduler/index.js 198 bytes [built] [code generated]
      ../../node_modules/scheduler/cjs/scheduler.production.min.js 4.14 KiB [built] [code generated]
  modules by path ./src/ 30.8 KiB (javascript) 80 bytes (css/mini-extract)
    ./src/main.tsx + 2 modules 30.8 KiB [built] [code generated]
    ./src/styles.css 50 bytes [built] [code generated]
    css ../../node_modules/css-loader/dist/cjs.js??ruleSet[1].rules[4].oneOf[8].use[1]!../../node_modules/postcss-loader/dist/cjs.js??ruleSet[1].rules[4].oneOf[8].use[2]!./src/styles.css 80 bytes [built] [code generated]
webpack 5.75.0 compiled successfully in 1656 ms

As can be seen, it is too verbose comparing to the previous:

Entrypoint main 161 KiB = runtime.8146301c839f438b.js 884 bytes main.2c4bd892d150ff6b.js 160 KiB
Entrypoint styles 992 bytes = runtime.8146301c839f438b.js 884 bytes styles.ef46db3751d8e999.css 0 bytes styles.b79f02fd2e89a8dd.js 108 bytes
chunk (runtime: runtime) main.2c4bd892d150ff6b.js (main) 174 KiB [initial] [rendered]
chunk (runtime: runtime) styles.ef46db3751d8e999.css, styles.b79f02fd2e89a8dd.js (styles) 50 bytes (javascript) 80 bytes (css/mini-extract) [initial] [rendered]
chunk (runtime: runtime) runtime.8146301c839f438b.js (runtime) 2.45 KiB [entry] [rendered]
webpack compiled successfully (2557329392266289)

 —————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————————

 >  NX   Successfully ran target build for project app (2s)

Expected Behavior

Should have a summarized console output

Related Issue(s)

Fixes #

@vercel
Copy link

vercel bot commented Feb 4, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated
nx-dev ⬜️ Ignored (Inspect) Feb 4, 2023 at 6:34PM (UTC)

@danr-za
Copy link
Contributor Author

danr-za commented Feb 6, 2023

@jaysoo @mandarini

Copy link
Member

@mandarini mandarini left a comment

Choose a reason for hiding this comment

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

Looks good to me. What. do you think @jaysoo ?

@mandarini mandarini merged commit 6748bc6 into nrwl:master Feb 7, 2023
@danr-za danr-za deleted the fix/webpack-stats-info branch February 7, 2023 16:41
@github-actions
Copy link

github-actions bot commented Mar 3, 2023

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants