Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

sourceMap.content "auto" option #3220

Closed
wants to merge 3 commits into from

Conversation

connorjclark
Copy link

@connorjclark connorjclark commented Jul 20, 2018

overview

Implements #3219

--source-map "content=auto"

Allows for easier composition of source maps.

If an input file specifies a sourceMappingURL comment, the source map will be fetched.
If there is no comment, given an input file of src/file.js, the map will be looked for
at src/file.js.map and src/file.map. Maps can fetched via HTTP too.

--source-map "content=auto,contents=file1.js*file1.js.map|file2.js*path/to/map.js.map|..."

Allows for overriding the auto resolution strategy.

other stuff

Open to a different format for sourceMap.contents.

This is a super set of the other source map content options. I suggest removing those in a new major version, and making this "auto" behavior the default.

MOCHA_GREP='locations over auto' SKIP_COMPRESS_TESTS=1 npm test

I needed a way to quickly run the new tests I added. Should be self explanatory.

@connorjclark
Copy link
Author

Can't get it passing on node v0.10. child_process.execSync does not exist in that version. Suggestions?

@connorjclark
Copy link
Author

Been using this in production for awhile. Any feedback from project maintainers?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant