react-native-reanimated icon indicating copy to clipboard operation
react-native-reanimated copied to clipboard

Fix CI issues with yarn

Open tjzel opened this issue 9 months ago • 1 comments

Summary

Due to changes to React Native app template, it uses Yarn 3.6.4 out of the box now. This version of Yarn cannot resolve packages from GitHub's pull request refs. Therefore we have to resolve it for it and install via a downloaded package. It's not something crazy - Yarn 1 would do exactly those steps.

Test plan

  • Test V8
  • Test monorepo build
  • Check static framework
  • Test build on Windows

Should not fail on the dependency installing steps with yarn. Ideally, they shouldn't fail at all.

tjzel avatar May 09 '24 13:05 tjzel

I fixed issues with regard to RN template using Yarn 3.6.4 by default.

@tomekzaw @piaskowyk Could you take a look at the failing V8 and monorepo CIs? I have the following questions.

  1. Does V8 CI fail because it doesn't yet support RN 0.74? Maybe you can see it on the spot, I haven't investigated it.

  2. What's exactly the purpose of the monorepo CI? It creates the following file structure:

.
└── monorepo
    ├── RootApp
    └── packages
        └── PackageApp

Which doesn't make a lot of sense - creating the RootApp step. Could you explain a bit your intentions behind this?

EDIT: Thanks for discussing it with me offline.

tjzel avatar May 09 '24 16:05 tjzel