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

Rename fork ReactSharedInternals -> ReactSharedInternalsClient #27767

Merged
merged 1 commit into from
Dec 1, 2023

Conversation

jackpope
Copy link
Contributor

Summary

Follow up from #27717 based on feedback to rename the fork module itself

How did you test this change?

  • yarn build
  • yarn test packages/scheduler/src/__tests__/SchedulerUMDBundle-test.internal.js

@facebook-github-bot facebook-github-bot added CLA Signed React Core Team Opened by a member of the React Core Team labels Nov 30, 2023
@react-sizebot
Copy link

Comparing: 60ad369...aca27d7

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 = 175.90 kB 175.90 kB = 54.75 kB 54.75 kB
oss-experimental/react-dom/cjs/react-dom.production.min.js = 177.97 kB 177.97 kB = 55.39 kB 55.39 kB
facebook-www/ReactDOM-prod.classic.js = 569.81 kB 569.81 kB = 100.29 kB 100.29 kB
facebook-www/ReactDOM-prod.modern.js = 553.67 kB 553.67 kB = 97.38 kB 97.38 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against aca27d7

Copy link
Member

@kassens kassens left a comment

Choose a reason for hiding this comment

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

ReactSharedClientInternals maybe reads a bit more natural?

@jackpope
Copy link
Contributor Author

@kassens Yeah, but the idea is to match the original module: https://github.com/facebook/react/blob/60ad3693a554f24b116f3f9afdb171fb4beb6d02/packages/react/src/ReactSharedInternalsClient.js

Which has Client/Server suffixed versions

Copy link
Collaborator

@sophiebits sophiebits left a comment

Choose a reason for hiding this comment

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

ty 🙏

@jackpope jackpope merged commit 3e97c00 into facebook:main Dec 1, 2023
36 checks passed
@jackpope jackpope deleted the rename-shared-internals branch December 1, 2023 14:48
EdisonVan pushed a commit to EdisonVan/react that referenced this pull request Apr 15, 2024
…ook#27767)

## Summary

Follow up from facebook#27717 based on feedback to rename the fork module itself

## How did you test this change?

- `yarn build`
- `yarn test
packages/scheduler/src/__tests__/SchedulerUMDBundle-test.internal.js`

Co-authored-by: Jack Pope <[email protected]>
bigfootjon pushed a commit that referenced this pull request Apr 18, 2024
## Summary

Follow up from #27717 based on feedback to rename the fork module itself

## How did you test this change?

- `yarn build`
- `yarn test
packages/scheduler/src/__tests__/SchedulerUMDBundle-test.internal.js`

Co-authored-by: Jack Pope <[email protected]>

DiffTrain build for commit 3e97c00.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed React Core Team Opened by a member of the React Core Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants