Skip to content

Minify both before and after browserify#736

Merged
petehunt merged 2 commits intofacebook:masterfrom
sophiebits:minify-before-browserify
Dec 29, 2013
Merged

Minify both before and after browserify#736
petehunt merged 2 commits intofacebook:masterfrom
sophiebits:minify-before-browserify

Conversation

@sophiebits
Copy link
Collaborator

Excludes ReactDefaultPerf and performanceNow from the minified build, so we save bytes. Looks like there are a few other minor differences because uglifyjs isn't quite idempotent but they appear to be harmless.

Feels silly but I don't think there's another way. (We need to run uglify after as well so that require, module, exports are minified)

   raw     gz Compared to master @ f877c6224fbf217d639a0ef2d7abf28c4cfbafa4
     =      = build/JSXTransformer.js
     =      = build/react-test.js
     =      = build/react-with-addons.js
  -406   -129 build/react-with-addons.min.js
     =      = build/react.js
  -406   -108 build/react.min.js

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why not just use uglifyify?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

because I didn't know about it!

petehunt added a commit that referenced this pull request Dec 29, 2013
Minify both before and after browserify
@petehunt petehunt merged commit eab2ede into facebook:master Dec 29, 2013
josephsavona added a commit that referenced this pull request May 15, 2024
* handle jsx fragments

* update effects for jsx fragments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants