fix(lib): rm comment, rename const => var

[michael-ciniawsky]

fixes #451 #450 , source-map-loader #34

[codecov]

Codecov Report

Merging #452 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff           @@
##           master    #452   +/-   ##
======================================
  Coverage    90.7%   90.7%           
======================================
  Files          10      10           
  Lines         398     398           
  Branches       85      85           
======================================
  Hits          361     361           
  Misses         37      37

Impacted Files	Coverage Δ
lib/convert-source-map.js	54.92% <ø> (ø)	✅
lib/css-base.js	97.14% <100%> (ø)	✅

---

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 0252643...0c77ea5. Read the comment docs.

[bebraw]

Ideally we could drop lib/convert-source-map.js as it was meant as a temporary solution.

[michael-ciniawsky]

@bebraw Is webpack #4458 solved ? The temporary fix was needed, because of problems with the convert-source-map dependency relying on require('fs') internally? I'm all in favour of removing it but how? 😛

[bebraw]

The problem with #4458 is that webpack might be doing the right thing here. No fs on web. But the bits we need don't rely on fs. I think we may have to wait Tobias to comment in order to know the right fix.

Merge this and push a patch version.