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

Bump fabric example to react-native@0.77.0-rc.3 #3277

Merged
merged 6 commits into from
Dec 18, 2024

Conversation

jakex7
Copy link
Member

@jakex7 jakex7 commented Dec 10, 2024

Description

Bumping FabricExample app to react-native@0.77.0-rc.3 to test with the next React Native version. I'll make all necessary changes to the package in the following PR.

@jakex7 jakex7 requested a review from j-piasecki December 10, 2024 14:07
j-piasecki pushed a commit that referenced this pull request Dec 13, 2024
## Description

* 54710a6 - Added `sourceSets` to the
package due to changes in ViewManager function signatures, which are no
longer nullable
* a20adca - Updated to reflect changes
in facebook/react-native#47551,
`findHostInstance_DEPRECATED` is now a property of the `default` object
instead of being a named export. Modified implementation reads from
default and fall back to named export as needed
* ac7e786 - Similar change to the
software-mansion/react-native-svg#2572 that uses
new API to create an association map between JS components and their
native implementations, avoiding `codegen` crawling through the entire
filesystem for `.mm` files.

## Test plan

App with React Native 0.77 (like
#3277)
should compile and work as expected.
Copy link
Member

@j-piasecki j-piasecki left a comment

Choose a reason for hiding this comment

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

Can you also quickly check the failing TS check? Let me know if you think it will take too long to solve.

FabricExample/package.json Outdated Show resolved Hide resolved
@jakex7
Copy link
Member Author

jakex7 commented Dec 17, 2024

Can you also quickly check the failing TS check? Let me know if you think it will take too long to solve.

Fixed in 121c498

EDIT: Now it probably will fail for Example 🤦‍♂️

EDIT2: Oh, they don’t share a common source, it should work.

@jakex7 jakex7 requested a review from j-piasecki December 17, 2024 14:23
@jakex7 jakex7 changed the title Bump fabric example to react-native@0.77.0-rc.1 Bump fabric example to react-native@0.77.0-rc.3 Dec 17, 2024
@j-piasecki j-piasecki merged commit 802af8a into main Dec 18, 2024
6 checks passed
@j-piasecki j-piasecki deleted the @jakex7/bumpFabricExample branch December 18, 2024 08:42
j-piasecki added a commit that referenced this pull request Dec 20, 2024
…3301)

## Description


#3278
and
#3277
added support for RN 0.77 where the result of generated java files has
changed but those weren't updated in the PRs. This should've been caught
by the CI but the `package.json` files weren't listed as paths that
would trigger it. This seems wrong, as I would imagine we would like the
CI to run when changing dependencies, especially React Native.

## Test plan

Build the example app (0.75 old arch) and Fabric example app (0.77, both
old and new arch)
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