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

Strip comments from the production build #5

Merged
merged 2 commits into from
Jul 17, 2016
Merged

Strip comments from the production build #5

merged 2 commits into from
Jul 17, 2016

Conversation

insin
Copy link
Contributor

@insin insin commented Jul 17, 2016

Do we also want to use Uglify's screw-ie8 option?

It will default to true as of 2.7.0, but Webpack is still on ~2.6.0.

If so, I can add to this PR - there are 3 different pieces of the uglification process it can be configured for - compress, mangle and output.

@ghost ghost added the CLA Signed label Jul 17, 2016
@gaearon
Copy link
Contributor

gaearon commented Jul 17, 2016

I’m fine with screw-ie8, we don’t officially support it in React anymore.

@vjeux
Copy link
Contributor

vjeux commented Jul 17, 2016

Lgtm, feel free to merge it when you're done with it

@ghost ghost added the CLA Signed label Jul 17, 2016
@vjeux vjeux merged commit b036bb9 into facebook:master Jul 17, 2016
@vjeux
Copy link
Contributor

vjeux commented Jul 17, 2016

Merging it right now so it doesn't introduce merge conflict hell if someone is working on the same part of the codebase. Feel free to send follow up pull requests

vjeux pushed a commit that referenced this pull request Jul 18, 2016
* Strip comments from the production build

* Add screw_ie8 options to UglifyJsPlugin
mikechau referenced this pull request in mikechau/create-react-app Jan 25, 2017
mikechau referenced this pull request in mikechau/create-react-app Jan 25, 2017
Remove non-text character from tsconfig.json closes #5
@lock lock bot locked and limited conversation to collaborators Jan 22, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants