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

feat: support React Native 0.76 #753

Merged
merged 44 commits into from
Oct 18, 2024
Merged

feat: support React Native 0.76 #753

merged 44 commits into from
Oct 18, 2024

Conversation

artus9033
Copy link
Contributor

@artus9033 artus9033 commented Sep 19, 2024

Summary

  • - align CMakeLists to account for merged RN SO libs
  • - updated both tester apps to 0.76-rc.6
  • - updated codegen rules to use hermes parser
  • - updated flow-remove-types to include necessary fixes for RN 76
  • - add missing fusebox related method to DevClient

Test plan

  • - tester app works with old/new arch
  • - tester federation works with old/new arch

Copy link

vercel bot commented Sep 19, 2024

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

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
repack-website ⬜️ Ignored (Inspect) Visit Preview Oct 18, 2024 10:30am

Copy link

changeset-bot bot commented Sep 19, 2024

🦋 Changeset detected

Latest commit: 4618622

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@callstack/repack Minor
@callstack/repack-dev-server Minor
@callstack/repack-init Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Member

@jbroma jbroma left a comment

Choose a reason for hiding this comment

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

@artus9033 thanks for the work on this! fantastic progress 🎉

we need to put 0.76 changes in Re.Pack in a separate PR

Let's make sure all of this is backwards compatible with 0.75 and 0.74 - did you manage to verify this already?

Other than that it looks solid 🎉

packages/repack/src/modules/DevServerClient.ts Outdated Show resolved Hide resolved
packages/repack/android/gradle.properties Outdated Show resolved Hide resolved
packages/repack/android/build.gradle Outdated Show resolved Hide resolved
packages/repack/android/CMakeLists.txt Outdated Show resolved Hide resolved
apps/tester-federation/package.json Outdated Show resolved Hide resolved
apps/tester-app/package.json Outdated Show resolved Hide resolved
apps/tester-app/package.json Outdated Show resolved Hide resolved
jbroma

This comment was marked as duplicate.

@artus9033 artus9033 changed the title chore: upgrade to RN 0.76.0-rc.1 chore: upgrade to RN 0.76.0-rc.2 Sep 29, 2024
@jbroma jbroma changed the title chore: upgrade to RN 0.76.0-rc.2 chore: upgrade to RN 0.76.0-rc.3 Oct 2, 2024
@jbroma
Copy link
Member

jbroma commented Oct 2, 2024

flow-types still being removed incorrectly, this PR to flow-remove-types should fix it:
facebook/flow#9213

@jbroma jbroma changed the title chore: upgrade to RN 0.76.0-rc.3 chore: upgrade to RN 0.76.0-rc.5 Oct 17, 2024
@jbroma
Copy link
Member

jbroma commented Oct 17, 2024

@jbroma jbroma changed the title chore: upgrade to RN 0.76.0-rc.5 chore: upgrade to RN 0.76.0-rc.6 Oct 17, 2024
@jbroma
Copy link
Member

jbroma commented Oct 17, 2024

patched RNScreens using changes from this PR: software-mansion/react-native-screens#2415, after that it builds correctly for Android 👍

all thats left is the flow issue and we should be g2g on this

@jbroma jbroma changed the title chore: upgrade to RN 0.76.0-rc.6 feat: support React Native 0.76 Oct 18, 2024
@jbroma jbroma merged commit 176324a into main Oct 18, 2024
5 checks passed
@jbroma jbroma deleted the chore/upgrade-to-rn-76-rc1 branch October 18, 2024 10:38
@jbroma
Copy link
Member

jbroma commented Oct 18, 2024

Thanks for your work on this @artus9033 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants