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.

Sign up for GitHub

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

Jump to bottom

upgrade uglifyjs2? #801

Closed

jrburke opened this issue Mar 30, 2015 · 1 comment

Milestone

Member

jrburke commented Mar 30, 2015

Check for latest release for the 2.1.17 release.

jrburke modified the milestones: 2.1.17, 2.1.18

jrburke modified the milestones: 2.1.19, 2.1.18

Member Author

jrburke commented May 29, 2015

The update seems fine, but some sourcemap tests are no longer matching, and have not had the time to dive into it more. Some notes for next attempt:

tests/lib/sourcemapSingle: file and sources look fine, but the mappings is different, confirm that it works correctly.
tests/lib/sourcemap/www-built: same as first item.
tests/lib/sourcemap/onejs: the "file" attribute in the .map file is the whole path, where it used to just be "built.js". Do not want full paths on the server showing up in the map file. Also confirm map is correct.

jrburke closed this as completed in

9ec41be

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment