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

SourceMapping pragma has changed to //# #213

Merged
merged 2 commits into from May 22, 2013

Conversation

Projects
None yet
2 participants
@mattrobenolt
Contributor

mattrobenolt commented May 21, 2013

See: https://docs.google.com/document/d/1U1RGAehQwRypUTovF1KRlpiOFze0b-_2gc6fAH0KY0k/edit

The spec was updated on May 16th since //@ was causing some issues with IE.

mishoo added a commit that referenced this pull request May 22, 2013

Merge pull request #213 from mattrobenolt/patch-1
SourceMapping pragma has changed to //#

@mishoo mishoo merged commit 2df4892 into mishoo:master May 22, 2013

1 check passed

default The Travis CI build passed
Details

Elergy pushed a commit to Elergy/grunt-contrib-uglify that referenced this pull request Jun 12, 2017

Updated to new sourceMappingURL syntax
The upstream specification has changed from //@ to //# which avoids IE
conditional comment compatibility issues and thus the need to wrap the
declaration in a /* */ comment.

See:
    mishoo/UglifyJS2#213 (comment)
    https://bugzilla.mozilla.org/show_bug.cgi?id=870361
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment