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

Builder needs some modernization #634

Closed
2 of 4 tasks
marcoscaceres opened this issue Mar 10, 2016 · 1 comment
Closed
2 of 4 tasks

Builder needs some modernization #634

marcoscaceres opened this issue Mar 10, 2016 · 1 comment

Comments

@marcoscaceres
Copy link
Member

We do a bunch of stuff in the builder that is now natively supported by r.js. We should just use r.js to do the following:

  • always default to uglyfy2.
  • Allow uglyfy2 to compress (with safe options).
  • use insertRequire instead of manually including require(["..."]) in the boilerplate.
  • remove boilerplate method, and use wrap: option instead.
@marcoscaceres
Copy link
Member Author

Modernized it a bit - now using uglyfy3. Eventually, want to move to webpack #1129

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

No branches or pull requests

1 participant