Skip to content

chore: bump main examples & lib to use [email protected] #2825

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

Merged
merged 24 commits into from
Jun 9, 2025

Conversation

kkafar
Copy link
Member

@kkafar kkafar commented Apr 2, 2025

Description

Seems that we do not need any library code changes this time.

New architecture

  • iOS works
  • Android seems to build fine

Old architecture

  • iOS works
  • Android seems to build fine

Problems

iOS new arch e2e fails - detox has some issues with reloading the app...

Changes

See commits.
I've mostly followed react native upgrade helper here + I've added .ruby-version files to our main examples, so that we the lock files generated by different maintainers are the same.

Test code and steps to reproduce

Checklist

@kkafar kkafar self-assigned this Apr 3, 2025
@kkafar
Copy link
Member Author

kkafar commented Apr 9, 2025

Nightly reanimated should already work -> @kkafar

@kkafar
Copy link
Member Author

kkafar commented Apr 16, 2025

I've disabled Test1649 for a while, because it causes crash during bundle loading.

image

The problem is resolved once the jotai dependency is removed.

We need to get rid of the lib or find a way to fix the issue.

@kkafar kkafar changed the title feat: support for RN 0.79 chore: bump FabricExample & lib to use [email protected] Apr 16, 2025
@kkafar kkafar changed the title chore: bump FabricExample & lib to use [email protected] chore: bump main examples & lib to use [email protected] Apr 16, 2025
@kkafar
Copy link
Member Author

kkafar commented Apr 16, 2025

I've reported the issue on detox repo here: wix/Detox#4760

@kkafar kkafar changed the title chore: bump main examples & lib to use [email protected] chore: bump main examples & lib to use [email protected] Jun 6, 2025
@kkafar kkafar marked this pull request as ready for review June 9, 2025 11:08
@kkafar kkafar merged commit ac7ae47 into main Jun 9, 2025
7 of 10 checks passed
@kkafar kkafar deleted the @kkafar/support-0.79.0-rc.4 branch June 9, 2025 19:05
kkafar added a commit that referenced this pull request Jun 9, 2025
## Description

Should be merged after #2825

After 0.80.0-rc.2 we do not require any library code changes. The apps
must be updated, but thats it.

> [!caution] 
I'll proceed here, ignoring the detox, so that we can work with recent
RN versions in our apps.

## Changes

Updated example apps following the upgrade helper.

## Test code and steps to reproduce

Apps should build just fine.

## Checklist

- [❌] Ensured that CI passes
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.

1 participant