fix(index): merge input and instrumented sourcemaps
#87
Conversation
Based on the work by @adamburgess in webpack-contrib#73 (comment). Added Webpack 3 tests and a fix for the sourcemaps file names Fix webpack-contrib#73
filipesilva
force-pushed
the
sourcemap-to-source
branch
from
0417300
to
90ec3ff
Compare
Node 4.8 jobs were removed because Webpack 4 does not support it.
Previous version showed a security warning for lodash on `npm run security`.
filipesilva
force-pushed
the
sourcemap-to-source
branch
from
90ec3ff
to
041bcaa
Compare
Codecov Report
@@ Coverage Diff @@
## master #87 +/- ##
==========================================
+ Coverage 92.30% 94.11% +1.80%
==========================================
Files 2 2
Lines 13 17 +4
Branches 2 3 +1
==========================================
+ Hits 12 16 +4
Misses 1 1
Continue to review full report at Codecov.
|
michael-ciniawsky
changed the title
input and instrumented sourcemaps
|
Heya @michael-ciniawsky, any update on when this might be merged/released? |
|
@d3viant0ne, @michael-ciniawsky any updates on this? |
|
I created a new npm package named coverage-istanbul-loader. It includes the fix mentioned above as well as other fixes and updated dependencies. |
|
Hey, @filipesilva, (just I'm happened with broken test after migration to Angular 8 and sourcemap is really broken :( ) |
Based on the work by @adamburgess in #73 (comment).
Added a minimal update to work with the latest webpack (used in CI tests), updated a dependency with a security issue,, fixed a test, and added a fix for the sourcemaps file names.
Type
Issues
SemVer