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

Upgrade rollup dependency #11591

Merged
merged 2 commits into from
Nov 18, 2017
Merged

Upgrade rollup dependency #11591

merged 2 commits into from
Nov 18, 2017

Conversation

misoguy
Copy link
Contributor

@misoguy misoguy commented Nov 18, 2017

Upgrade rollup dependency to finish up with #11456.
It seems that most of the build shows a little decrease in size after upgrading rollup.

@gaearon
Copy link
Collaborator

gaearon commented Nov 18, 2017

Can you post a diff of build/facebook-www/ReactDOM-prod.js before and after the upgrade?

@misoguy
Copy link
Contributor Author

misoguy commented Nov 18, 2017

How should I post the diff? Is a screenshot enough or should I post the code somewhere?

@misoguy
Copy link
Contributor Author

misoguy commented Nov 18, 2017

[Left] After rollup upgrade (null; is gone)
[Right] Before rollup upgrade
image

@gaearon gaearon merged commit 01a867b into facebook:master Nov 18, 2017
Ethan-Arrowood pushed a commit to Ethan-Arrowood/react that referenced this pull request Dec 8, 2017
* Record build results before upgrading rollup

* Upgrade rollup and record new results.json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants