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): extract css breaks hmr and source-maps #5906

Merged
merged 1 commit into from
Jun 14, 2019
Merged

Conversation

manniL
Copy link
Member

@manniL manniL commented Jun 10, 2019

Resolves #5386 #5390

Types of changes

  • Bug fix (a non-breaking change which fixes an issue)

Description

Related: faceyspacey/extract-css-chunks-webpack-plugin#161

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly. (PR: #)
  • I have added tests to cover my changes (if not applicable, please state why)
  • All new and existing tests are passing.

@galvez
Copy link

galvez commented Jun 10, 2019

@manniL thanks for handling this, I think it explains some oddity I've seen.

@manniL
Copy link
Member Author

manniL commented Jun 10, 2019

Works fine ☺️

@galvez Np at all! 👍

Copy link
Member

@Atinux Atinux left a comment

Choose a reason for hiding this comment

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

Nice one @manniL

@manniL manniL requested a review from pi0 June 11, 2019 18:37
@pi0 pi0 changed the title fix: extract css hmr fix(webpack): extract css breaks hmr and source-maps Jun 14, 2019
@pi0 pi0 merged commit 3d80ef4 into dev Jun 14, 2019
@pi0 pi0 deleted the fix/extract-css-hmr branch June 14, 2019 17:33
@pi0 pi0 mentioned this pull request Jun 15, 2019
@danielroe danielroe added the 2.x label Jan 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Setting 'extractCSS' to 'true' invalidates HMR
5 participants