Skip to content
This repository has been archived by the owner on Dec 5, 2019. It is now read-only.

Warning output missed important information #281

Closed
sokra opened this issue Apr 26, 2018 · 0 comments · Fixed by #306
Closed

Warning output missed important information #281

sokra opened this issue Apr 26, 2018 · 0 comments · Fixed by #306

Comments

@sokra
Copy link
Member

sokra commented Apr 26, 2018

After upgrading the uglifyjs-webpack-plugin dependency in webpack I noticied that the warnings format changed:

test result

  • Before: single warning for an asset, After: each warning separate
  • Before: info about the asset, After: information missing
  • Before: note about UglifyJS, After: no info

I propose to merge warnings by module and add a not about asset and uglifyjs-webpack-plugin. Example:

WARNING in ./index.js in bundle.js (uglifyjs-webpack-plugin)
6,0: Dropping side-effect-free statement
8,0: Dropping unused function someUnUsedFunction1
9,0: Dropping unused function someUnUsedFunction2
10,0: Dropping unused function someUnUsedFunction3
11,0: Dropping unused function someUnUsedFunction4
12,0: Dropping unused function someUnUsedFunction5

WARNING in ./a.js in bundle.js (uglifyjs-webpack-plugin)
3,0: Dropping unused function someRemoteUnUsedFunction1
4,0: Dropping unused function someRemoteUnUsedFunction2
5,0: Dropping unused function someRemoteUnUsedFunction3
6,0: Dropping unused function someRemoteUnUsedFunction4
7,0: Dropping unused function someRemoteUnUsedFunction5
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants