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 RN_FB bundles for react-is #23101

Merged
merged 3 commits into from
Jan 12, 2022
Merged

Add RN_FB bundles for react-is #23101

merged 3 commits into from
Jan 12, 2022

Conversation

motiz88
Copy link
Contributor

@motiz88 motiz88 commented Jan 12, 2022

Summary

Adds react-is build artifacts for use with React Native within Meta.

How did you test this change?

  • Observe new bundles being written in the CI job on this PR.

@sizebot
Copy link

sizebot commented Jan 12, 2022

Comparing: 9a7e6bf...79aff1b

Critical size changes

Includes critical production bundles, as well as any change greater than 2%:

Name +/- Base Current +/- gzip Base gzip Current gzip
oss-stable/react-dom/cjs/react-dom.production.min.js = 129.52 kB 129.52 kB = 41.53 kB 41.53 kB
oss-experimental/react-dom/cjs/react-dom.production.min.js = 134.65 kB 134.65 kB = 43.01 kB 43.01 kB
facebook-www/ReactDOM-prod.classic.js = 427.76 kB 427.76 kB = 78.53 kB 78.53 kB
facebook-www/ReactDOM-prod.modern.js = 417.45 kB 417.45 kB = 77.06 kB 77.06 kB
facebook-www/ReactDOMForked-prod.classic.js = 427.76 kB 427.76 kB = 78.53 kB 78.53 kB
facebook-react-native/react-is/cjs/ReactIs-dev.js +∞% 0.00 kB 7.96 kB +∞% 0.00 kB 2.11 kB
facebook-react-native/react-is/cjs/ReactIs-prod.js +∞% 0.00 kB 5.58 kB +∞% 0.00 kB 1.36 kB
facebook-react-native/react-is/cjs/ReactIs-profiling.js +∞% 0.00 kB 5.58 kB +∞% 0.00 kB 1.36 kB

Significant size changes

Includes any change greater than 0.2%:

Expand to show
Name +/- Base Current +/- gzip Base gzip Current gzip
facebook-react-native/react-is/cjs/ReactIs-dev.js +∞% 0.00 kB 7.96 kB +∞% 0.00 kB 2.11 kB
facebook-react-native/react-is/cjs/ReactIs-prod.js +∞% 0.00 kB 5.58 kB +∞% 0.00 kB 1.36 kB
facebook-react-native/react-is/cjs/ReactIs-profiling.js +∞% 0.00 kB 5.58 kB +∞% 0.00 kB 1.36 kB

Generated by 🚫 dangerJS against 79aff1b

@motiz88 motiz88 marked this pull request as ready for review January 12, 2022 19:26
@motiz88 motiz88 merged commit c09596c into facebook:main Jan 12, 2022
@motiz88 motiz88 deleted the react-is-rn-fb branch January 12, 2022 21:53
@gaearon
Copy link
Collaborator

gaearon commented Jan 13, 2022

why do we want to use it? i think whenever we look at use cases for it, there are usually better approaches that don't involve introspecting types.

@motiz88
Copy link
Contributor Author

motiz88 commented Jan 13, 2022

@gaearon This isn't for any new use cases (that I'm aware of) - it's strictly for using a single version that is in sync with our build of React at existing call sites (mostly in third-party packages, but a handful in first-party code) that are currently relying pretty arbitrarily on a mix of react-is@16 and react-is@17 from npm; also for tests via react-test-renderer, which we already vendor. See D33481964 and D33483002 for the discussion.

facebook-github-bot pushed a commit to facebook/react-native that referenced this pull request Jan 19, 2022
Summary:
This sync includes the following changes:
- **[51947a14b](facebook/react@51947a14b )**: Refactored how React/DevTools log Timeline performance data ([#23102](facebook/react#23102)) //<Brian Vaughn>//
- **[c09596cc6](facebook/react@c09596cc6 )**: Add RN_FB bundles for react-is ([#23101](facebook/react#23101)) //<Moti Zilberman>//
- **[9a7e6bf0d](facebook/react@9a7e6bf0d )**: Add --no-show-signature to "git show" commands ([#23038](facebook/react#23038)) //<Stefan Sundin>//
- **[2f26eb85d](facebook/react@2f26eb85d )**: Add exports field to react-refresh's package.json ([#23087](facebook/react#23087)) //<Gray Zhang>//
- **[811634762](facebook/react@811634762 )**: add enableTransitionTracing feature flag ([#23079](facebook/react#23079)) //<Luna Ruan>//

Changelog:
[General][Changed] - React Native sync for revisions fe905f1...51947a1

jest_e2e[run_all_tests]

Reviewed By: rickhanlonii, kacieb

Differential Revision: D33634332

fbshipit-source-id: a83b663a122a2cb79225ca33a007fe1774728c03
zhengjitf pushed a commit to zhengjitf/react that referenced this pull request Apr 15, 2022
* Add RN_FB bundles for react-is

* Update packaging.js

* Add ReactNativeInternalFeatureFlags to externals
nevilm-lt pushed a commit to nevilm-lt/react that referenced this pull request Apr 22, 2022
* Add RN_FB bundles for react-is

* Update packaging.js

* Add ReactNativeInternalFeatureFlags to externals
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