This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit fdc8931:

Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/46557/

Node 6 failed but it's expected because WASM support was introduced in Node 8

Unfortunately we cannot drop Node.js 6 until Babel 8 😕

@nicolo-ribaudo yep, I know.

I also maintain source-map-js which is the fork of source-map, but with algorithmic optimizations. It's 4x faster than source-map@6, but still 2x slower than source-map-sync.

So I think we need to make a choice :)

@7rulnik Does source-map-js supports node 6? It makes sense to migrate to it first and the see if other improvements are necessary.

@TrySound yeah, it should. Will update this PR in few days

Hi @nicolo-ribaudo are you interested in merging this ?

@7rulnik can you update the dependency to ^1.0.0 ?

This will also fix an annoying issue in VS Code.

source-map@<0.7.1 has an invalid typings field in its package.json which is causing VS Code to report an error that it can't find the type declarations. This is fixed in source-map@>=0.7.1. Moving to source-map-js would eliminate the issue altogether.

All of the SourceMapConsumers have switched over to @ampproject/remapping and @jridgewell/trace-mapping. There are still some SourceMapGenerators that could be switched to source-map-js.

@jridgewell yeah. I can update PR if it makes sense and we will merge it