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

Add UMD production+profiling entry points #13642

Merged
merged 5 commits into from
Sep 14, 2018

Conversation

bvaughn
Copy link
Contributor

@bvaughn bvaughn commented Sep 13, 2018

Resolves #13634 by enabling apps that use the UMD build of React (and co) to use the new profiling and tracing APIs in production mode.

Note that this PR builds on top of PR #13641. The only commit this branch contains is 18b4a27.

Changes

  • Removed unnecessary CJS profiling bundle for react package (since it's not necessary).
  • Added new UMD_PROFILING bundle type and plugged into react and react-dom.
  • Added new umd/schedule.profiling.min.js shim to schedule package. (This is just a copy of umd/schedule.production.min.js.)

@sizebot
Copy link

sizebot commented Sep 13, 2018

Details of bundled changes.

Comparing: 4bcee56...80ffb58

schedule

File Filesize Diff Gzip Diff Prev Size Current Size Prev Gzip Current Gzip ENV
schedule.development.js n/a n/a 0 B 19.17 KB 0 B 5.74 KB UMD_DEV
schedule.production.min.js n/a n/a 0 B 3.16 KB 0 B 1.53 KB UMD_PROD

react

File Filesize Diff Gzip Diff Prev Size Current Size Prev Gzip Current Gzip ENV
react.profiling.min.js n/a n/a 0 B 11.69 KB 0 B 4.49 KB UMD_PROFILING

react-dom

File Filesize Diff Gzip Diff Prev Size Current Size Prev Gzip Current Gzip ENV
react-dom.profiling.min.js n/a n/a 0 B 95.15 KB 0 B 30.79 KB UMD_PROFILING

Generated by 🚫 dangerJS

Copy link
Collaborator

@gaearon gaearon left a comment

Choose a reason for hiding this comment

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

Should we add the new Schedule bundle to the test that verifies matching exports?

@bvaughn
Copy link
Contributor Author

bvaughn commented Sep 14, 2018

Whoops, yeah. Thanks for catching that.

@bvaughn bvaughn merged commit 8bc0bca into facebook:master Sep 14, 2018
@bvaughn bvaughn deleted the add-umd-profiling-entry-points branch September 14, 2018 00:44
@@ -49,6 +50,16 @@ ${license}
${source}`;
},

/***************** UMD_PROFILING *****************/
Copy link
Contributor

Choose a reason for hiding this comment

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

Seems more '*' than others 😄

Simek pushed a commit to Simek/react that referenced this pull request Oct 25, 2018
* Added UMD_PROFILING type to react-dom and scheduling package. Added UMD shim to schedule package.
* Added new schedule umd prod+prof bundle to API test
jetoneza pushed a commit to jetoneza/react that referenced this pull request Jan 23, 2019
* Added UMD_PROFILING type to react-dom and scheduling package. Added UMD shim to schedule package.
* Added new schedule umd prod+prof bundle to API test
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.

5 participants