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

Require min style in both places. Fixes #2997 #2998

Merged

tmcw merged 1 commit into master from 2997-one-true-style

Aug 12, 2016

Contributor

tmcw commented Aug 12, 2016 •

edited by lucaswoj

This requires latest.min in both the validation and spec code, thus eliminating dead code.

Before: du -sh dist/mapbox-gl.js -> 512K
After: du -sh dist/mapbox-gl.js -> 476K

Checklist

briefly describe the changes in this PR
write tests for all new functionality
document any changes or additions to public APIs
post scores for all benchmarks on your branch and master
do a quick sanity check on the debug page
get a PR review 👍 / 👎


          Require min style in both places. Fixes #2997

63ab85f

Contributor

lucaswoj commented Aug 12, 2016

Good catch! Thank you @tmcw.

tmcw merged commit 50142de into master

tmcw deleted the 2997-one-true-style branch

August 12, 2016 20:32

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