Codecov Report

Merging #280 into master will increase coverage by 0.24%.
The diff coverage is 93.75%.

@@            Coverage Diff             @@
##           master     #280      +/-   ##
==========================================
+ Coverage   92.83%   93.08%   +0.24%     
==========================================
  Files           5        5              
  Lines         405      405              
  Branches      152      153       +1     
==========================================
+ Hits          376      377       +1     
+ Misses         28       27       -1     
  Partials        1        1              

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 ede8439...791db21. Read the comment docs.

Sorry, we can't accept this PR, because webpack and other modules uses old sync version of source-map, also it is used only for errors, so no big perf here

because webpack and other modules uses old sync version of source-map

I thought of that, but since there is nothing breaking, I made this PR. I thought may be other packages could update it one at a time.

also it is used only for errors, so no big perf here

I am not completely aware of the internals. We have quite a large app, and minify phase is quite slow, just felt like any improvement is good to have 😅 .

I thought of that, but since there is nothing breaking, I made this PR. I thought may be other packages could update it one at a time.

We want to do it together with all future codebase - future

I am not completely aware of the internals. We have quite a large app, and minify phase is quite slow, just felt like any improvement is good to have sweat_smile .

Updating source map do not improve perf in this case 😄