close
The Wayback Machine - https://web.archive.org/web/20201020043054/https://github.com/facebook/react/pull/20012
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

Fixture: Legacy JSX Runtimes #20012

Merged
merged 2 commits into from Oct 14, 2020
Merged

Fixture: Legacy JSX Runtimes #20012

merged 2 commits into from Oct 14, 2020

Conversation

@gaearon
Copy link
Member

@gaearon gaearon commented Oct 13, 2020

Adds checked-in bundles for JSX runtimes for different stable React versions. Those were edited by hand.
Each comes with an integration testing suite adjusted for that version.

They look like this:

Screenshot 2020-10-14 at 00 16 37

cd fixtures/legacy-jsx-runtimes 
yarn
yarn test

Screenshot 2020-10-14 at 00 17 23

The regression suite contains a lint step and some hand-picked tests adapted from different existing ones in the repo. We run in three configurations: jsxDEV in DEV, jsx in DEV, and jsx in PROD.

Differences Between Bundles

Each lower bundle is based on the higher one.

  • 16: Use old stacks. Remove object-assign from the output.
  • 15: Read old stacks from ReactComponentTreeHook.
  • 14: Remove stacks. Use instance.getName() in messages. Read ReactCurrentOwner from react/lib/.
@codesandbox
Copy link

@codesandbox codesandbox bot commented Oct 13, 2020

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit a58529b:

Sandbox Source
React Configuration
@sizebot
Copy link

@sizebot sizebot commented Oct 13, 2020

No significant bundle size changes to report.

Size changes (experimental)

Generated by 🚫 dangerJS against a58529b

@sizebot
Copy link

@sizebot sizebot commented Oct 13, 2020

No significant bundle size changes to report.

Size changes (stable)

Generated by 🚫 dangerJS against a58529b

Copy link
Member

@rickhanlonii rickhanlonii left a comment

Great stuff!!

@gaearon gaearon merged commit 6d50a9d into facebook:master Oct 14, 2020
2 of 10 checks passed
2 of 10 checks passed
ci/circleci: yarn_build CircleCI is running your tests
Details
ci/circleci: yarn_test CircleCI is running your tests
Details
ci/circleci: yarn_test_prod CircleCI is running your tests
Details
ci/circleci: yarn_test_prod_www CircleCI is running your tests
Details
ci/circleci: yarn_test_prod_www_variant CircleCI is running your tests
Details
ci/circleci: yarn_test_www CircleCI is running your tests
Details
ci/circleci: yarn_test_www_variant CircleCI is running your tests
Details
ci/codesandbox Building packages...
Details
Facebook CLA Check Contributor License Agreement is valid!
Details
ci/circleci: setup Your tests passed on CircleCI!
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.