Skip to content

Fixed invalid chunkFileName#214

Merged
shellscape merged 1 commit intoshellscape:masterfrom
zauan:fix/webpack-5-hotUpdateChunkFilename
Mar 13, 2021
Merged

Fixed invalid chunkFileName#214
shellscape merged 1 commit intoshellscape:masterfrom
zauan:fix/webpack-5-hotUpdateChunkFilename

Conversation

@zauan
Copy link
Contributor

@zauan zauan commented Feb 22, 2021

This PR contains:

  • bugfix
  • feature
  • refactor
  • tests
  • documentation
  • metadata

Breaking Changes?

  • yes
  • no

If yes, please describe the breakage.

Please Describe Your Changes

As per Webpack documentation, [runtime] can be used only for hotUpdateMainFilename and not hotUpdateChunkFilename ( https://webpack.js.org/configuration/output/#outputhotupdatechunkfilename ). By having this template string inside outputhotupdatechunkfilename , the HMR breaks as chunk file names will have an incorrect name and the files won't be found on the server

As per Webpack documentation, [runtime] can be used only for hotUpdateMainFilename and not hotUpdateChunkFilename ( https://webpack.js.org/configuration/output/#outputhotupdatechunkfilename ). By having this template string inside outputhotupdatechunkfilename , the HMR breaks
Copy link
Contributor

@bz2 bz2 left a comment

Choose a reason for hiding this comment

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

Thanks! Also hit this issue after upgrading, and have verified locally that this change resolves the broken urls.

@shellscape shellscape merged commit 97288ad into shellscape:master Mar 13, 2021
@shellscape
Copy link
Owner

thanks!

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

Successfully merging this pull request may close these issues.

3 participants