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: RN 0.72 Upgrade #902

Merged
merged 33 commits into from
Dec 12, 2023
Merged

feat: RN 0.72 Upgrade #902

merged 33 commits into from
Dec 12, 2023

Conversation

oscb
Copy link
Contributor

@oscb oscb commented Dec 5, 2023

RN 0.72 upgrade

This is a full upgrade of the monorepo to support RN 0.72+ for testing/development.

Added another 0.73 App for testing (not working with E2E tests yet).

All devDependencies have been upgraded: Yarn, TS, Jest, ESLint, Detox, etc

Some changes:

  • New shared private package for holding common dev utils
  • E2E and Example are now split between 2 different apps
  • Tests are not included in the built package anymore
  • The E2E tests against release version of the app, also refactored not to use hacks for URL replacement but proxy options.

Known issues:

  • Detox is not running properly in GHA. It runs locally fine.
    - TSJest marks non-errors on individual packages (e.g. Argument of type lib.SegmentClient is not assignable to src.SegmentClient) due to configurations at individual level. Monorepo level testing works properly

Copy link
Contributor

@alanjcharles alanjcharles left a comment

Choose a reason for hiding this comment

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

great work man!

.github/workflows/ci.yml Outdated Show resolved Hide resolved
Comment on lines +41 to +42
this.consentCategoryProvider = consentCategoryProvider;
this.categories = categories;

Choose a reason for hiding this comment

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

🤞

Copy link

@chrisradek chrisradek left a comment

Choose a reason for hiding this comment

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

Thanks for doing this work - had a couple questions, and the .only that was left in, but otherwise looks good!

Copy link

@chrisradek chrisradek left a comment

Choose a reason for hiding this comment

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

Did upgrading detox help the e2e in ci situation?

@oscb
Copy link
Contributor Author

oscb commented Dec 12, 2023

Did upgrading detox help the e2e in ci situation?

@chrisradek sadly no, I'll try something else

@oscb oscb merged commit 3dafa18 into master Dec 12, 2023
8 of 10 checks passed
@oscb oscb deleted the rnupgrade branch December 12, 2023 20:44
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.

3 participants