Closed FengXianGuo closed 5 years ago
Merging #251 into master will not change coverage. The diff coverage is
100%
.
@@ Coverage Diff @@
## master #251 +/- ##
=======================================
Coverage 94.81% 94.81%
=======================================
Files 25 25
Lines 733 733
=======================================
Hits 695 695
Misses 38 38
Impacted Files | Coverage Δ | |
---|---|---|
lib/WebpackUtils.js | 94.36% <100%> (ø) |
:arrow_up: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 3f3fed2...4da69da. Read the comment docs.
Thanks for the contribution! Let me pull it down and run the examples.
Confirmed that the fix works. I'll be following up with a patch that adds an example of hard-source targeting webpack 3 and 4 to guard against future regressions.
Yeah, thanks for merging 😎
Released 5.0.1 on npm with the patch.
throw signature changed error when running hard-source-webpack-plugin with happypack
here the issues: https://github.com/mzgoddard/hard-source-webpack-plugin/issues/326 https://github.com/mzgoddard/hard-source-webpack-plugin/issues/439