shellscape / webpack-manifest-plugin

webpack plugin for generating asset manifests
MIT License
1.44k stars 184 forks source link

Manifest file is not taking compressed files #231

Closed jkevingutierrez closed 3 years ago

jkevingutierrez commented 3 years ago

Expected Behavior

Using compression-webpack-plugin with webpack-manifest-plugin should include the compressed files generated by compression-webpack-plugin in the manifest.json file

{
  "input.js": "input.js",
  "input.js.gz": "input.js.gz"
}

Actual Behavior

Files generated by compression-webpack-plugin are not included in the manifest.json file

{
  "input.js": "input.js"
}

Additional Information

It was working before on Webpack v4 and manifest plugin v2: https://repl.it/@jkevingutierrez/manifest-plugin-repro-1#dist/manifest.json

shellscape commented 3 years ago

See #229 for the auto issue ( and please search issues )

Did this work in the previous version of this plugin on webpack v4?

jkevingutierrez commented 3 years ago

(My bad, I just searched in open issues)

About the compression problem, it was working before on Webpack v4:

https://repl.it/@jkevingutierrez/manifest-plugin-repro-1#dist/manifest.json

shellscape commented 3 years ago

No problem. I'll use your repro to make an integration test and work on this

shellscape commented 3 years ago

Unfortunately you're running into an early adoption problem :( Webpack v5 just isn't ready, and the official plugins aren't either. Please see this comment https://github.com/webpack/webpack/issues/11822#issuecomment-738121628 in a webpack core issue. They have some more work to do in order to get this to work like it did with Webpack v4.

stale[bot] commented 3 years ago

Hey folks. This issue hasn't received any traction for 60 days, so we're going to close this for housekeeping. If this is still an ongoing issue, please do consider contributing a Pull Request to resolve it. Further discussion is always welcome even with the issue closed. If anything actionable is posted in the comments, we'll consider reopening it.