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

Build react-dom in builds/facebook-fbsource #30711

Merged
merged 3 commits into from
Sep 18, 2024

Conversation

yungsters
Copy link
Contributor

@yungsters yungsters commented Aug 15, 2024

Summary

Builds react-dom for React Native so that it also populates the builds/facebook-fbsource branch.

NOTE: For Meta employees, D61354219 is the internal integration.

How did you test this change?

$ yarn build
…
$ ls build/facebook-react-native/react-dom/cjs
ReactDOM-dev.js       ReactDOM-prod.js      ReactDOM-profiling.js

Copy link

vercel bot commented Aug 15, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
react-compiler-playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 18, 2024 8:27pm

@react-sizebot
Copy link

react-sizebot commented Aug 15, 2024

Comparing: 09d8283...d5f26fb

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.js = 6.68 kB 6.68 kB = 1.83 kB 1.82 kB
oss-stable/react-dom/cjs/react-dom-client.production.js = 508.22 kB 508.22 kB = 90.95 kB 90.95 kB
oss-experimental/react-dom/cjs/react-dom.production.js = 6.69 kB 6.69 kB = 1.83 kB 1.83 kB
oss-experimental/react-dom/cjs/react-dom-client.production.js = 513.15 kB 513.15 kB = 91.67 kB 91.67 kB
facebook-www/ReactDOM-prod.classic.js = 602.57 kB 602.57 kB = 106.64 kB 106.64 kB
facebook-www/ReactDOM-prod.modern.js = 578.84 kB 578.84 kB = 102.76 kB 102.76 kB
facebook-react-native/react-dom/cjs/ReactDOM-dev.js +∞% 0.00 kB 17.15 kB +∞% 0.00 kB 3.78 kB
facebook-react-native/react-dom/cjs/ReactDOM-prod.js +∞% 0.00 kB 6.48 kB +∞% 0.00 kB 1.81 kB
facebook-react-native/react-dom/cjs/ReactDOM-profiling.js +∞% 0.00 kB 6.48 kB +∞% 0.00 kB 1.81 kB
facebook-react-native/react-dom/cjs/ReactDOMClient-dev.js +∞% 0.00 kB 974.86 kB +∞% 0.00 kB 164.39 kB
facebook-react-native/react-dom/cjs/ReactDOMClient-prod.js +∞% 0.00 kB 533.29 kB +∞% 0.00 kB 94.87 kB
facebook-react-native/react-dom/cjs/ReactDOMClient-profiling.js +∞% 0.00 kB 558.22 kB +∞% 0.00 kB 98.62 kB
facebook-react-native/react-dom/cjs/ReactDOMProfiling-dev.js +∞% 0.00 kB 991.20 kB +∞% 0.00 kB 167.21 kB
facebook-react-native/react-dom/cjs/ReactDOMProfiling-prod.js +∞% 0.00 kB 538.79 kB +∞% 0.00 kB 95.95 kB
facebook-react-native/react-dom/cjs/ReactDOMProfiling-profiling.js +∞% 0.00 kB 564.16 kB +∞% 0.00 kB 99.77 kB
facebook-react-native/react-dom/cjs/ReactDOMTestUtils-dev.js +∞% 0.00 kB 1.39 kB +∞% 0.00 kB 0.71 kB
facebook-react-native/react-dom/cjs/ReactDOMTestUtils-prod.js +∞% 0.00 kB 0.67 kB +∞% 0.00 kB 0.41 kB
facebook-react-native/react-dom/cjs/ReactDOMTestUtils-profiling.js +∞% 0.00 kB 0.67 kB +∞% 0.00 kB 0.41 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-dom/cjs/ReactDOM-dev.js +∞% 0.00 kB 17.15 kB +∞% 0.00 kB 3.78 kB
facebook-react-native/react-dom/cjs/ReactDOM-prod.js +∞% 0.00 kB 6.48 kB +∞% 0.00 kB 1.81 kB
facebook-react-native/react-dom/cjs/ReactDOM-profiling.js +∞% 0.00 kB 6.48 kB +∞% 0.00 kB 1.81 kB
facebook-react-native/react-dom/cjs/ReactDOMClient-dev.js +∞% 0.00 kB 974.86 kB +∞% 0.00 kB 164.39 kB
facebook-react-native/react-dom/cjs/ReactDOMClient-prod.js +∞% 0.00 kB 533.29 kB +∞% 0.00 kB 94.87 kB
facebook-react-native/react-dom/cjs/ReactDOMClient-profiling.js +∞% 0.00 kB 558.22 kB +∞% 0.00 kB 98.62 kB
facebook-react-native/react-dom/cjs/ReactDOMProfiling-dev.js +∞% 0.00 kB 991.20 kB +∞% 0.00 kB 167.21 kB
facebook-react-native/react-dom/cjs/ReactDOMProfiling-prod.js +∞% 0.00 kB 538.79 kB +∞% 0.00 kB 95.95 kB
facebook-react-native/react-dom/cjs/ReactDOMProfiling-profiling.js +∞% 0.00 kB 564.16 kB +∞% 0.00 kB 99.77 kB
facebook-react-native/react-dom/cjs/ReactDOMTestUtils-dev.js +∞% 0.00 kB 1.39 kB +∞% 0.00 kB 0.71 kB
facebook-react-native/react-dom/cjs/ReactDOMTestUtils-prod.js +∞% 0.00 kB 0.67 kB +∞% 0.00 kB 0.41 kB
facebook-react-native/react-dom/cjs/ReactDOMTestUtils-profiling.js +∞% 0.00 kB 0.67 kB +∞% 0.00 kB 0.41 kB

Generated by 🚫 dangerJS against 2531d13

@yungsters
Copy link
Contributor Author

Hmm… actually, ReactDOMFB.js is pretty specific to www (e.g. ReactFiberErrorDialogWWW).

I'm going to switch this to build the non-FB react-dom for facebook-react-native.

@yungsters
Copy link
Contributor Author

Let me rebase this.

Copy link
Member

@rickhanlonii rickhanlonii left a comment

Choose a reason for hiding this comment

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

LGTM

@yungsters yungsters merged commit e72127a into facebook:main Sep 18, 2024
187 checks passed
@yungsters yungsters deleted the react-dom branch September 18, 2024 21:44
jackpope added a commit that referenced this pull request Sep 19, 2024
Commit artifact actions are breaking after
#30711

See:
https://github.com/facebook/react/actions/runs/10930658977/job/30344033974

> mv: cannot stat 'build/facebook-react-native/react/dom/': No such file
or directory

After build, the new artifacts are in `/react-dom/cjs`, not
`/react/dom/`
```
$> yarn build
$> ls build/facebook-react-native/react/
# ... no dom
$> ls build/facebook-react-native/react-dom/cjs
```
github-actions bot pushed a commit that referenced this pull request Sep 19, 2024
Commit artifact actions are breaking after
#30711

See:
https://github.com/facebook/react/actions/runs/10930658977/job/30344033974

> mv: cannot stat 'build/facebook-react-native/react/dom/': No such file
or directory

After build, the new artifacts are in `/react-dom/cjs`, not
`/react/dom/`
```
$> yarn build
$> ls build/facebook-react-native/react/
# ... no dom
$> ls build/facebook-react-native/react-dom/cjs
```

DiffTrain build for commit e495392.
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.

4 participants