Skip to content

[Monorepo] Fix yarn validation CI#3524

Merged
m-bert merged 6 commits into
@mbert/monorepofrom
@mbert/monorepo-fix-yarn-validation
May 22, 2025
Merged

[Monorepo] Fix yarn validation CI#3524
m-bert merged 6 commits into
@mbert/monorepofrom
@mbert/monorepo-fix-yarn-validation

Conversation

@m-bert
Copy link
Copy Markdown
Collaborator

@m-bert m-bert commented May 22, 2025

Description

yarn validation CI performs yarn install in all changed directories that contain package.json. However, we have some directories which should not undergo this procedure:

  • DrawerLayout
  • ReanimatedDrawerLayout
  • Swipeable
  • ReanimatedSwipeable
  • jest-utils

Test plan

Hopefully CI on main monorepo PR passes

@m-bert m-bert changed the base branch from main to @mbert/monorepo May 22, 2025 12:38
@m-bert m-bert marked this pull request as ready for review May 22, 2025 12:38
@m-bert m-bert requested a review from tjzel May 22, 2025 12:38
Copy link
Copy Markdown
Contributor

@tjzel tjzel left a comment

Choose a reason for hiding this comment

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

LGTM

@m-bert m-bert merged commit d7279a6 into @mbert/monorepo May 22, 2025
1 check passed
@m-bert m-bert deleted the @mbert/monorepo-fix-yarn-validation branch May 22, 2025 13:11
@m-bert m-bert mentioned this pull request May 22, 2025
23 tasks
m-bert added a commit that referenced this pull request May 23, 2025
## Description

This is base PR that migrates our repository into monorepo structure. It creates following structure:

- `/apps` - here we will put our example apps.
- `/packages` - main directory for Gesture Handler package, along with documentation.
- `/scripts` - scripts that are used by _**GestureHandler**_ (for example formatting scripts)

## Steps

<details>
<summary>Here is the list of PRs that fix issues caused by changing structure of the repository:</summary>

1. #3487
2. #3490
3. #3494
4. #3495
5. #3496
6. #3497
7. #3503
8. #3506
9. #3507
10. #3510
11. #3515
12. #3518
13. #3519
14. #3524
15. #3525
16. #3526
18. #3528

</details>

Note that most of them depends on previous ones, so **_they had to be merged in given order_** (at least up to no. 9).

## Test plan

<details>
<summary>Following subjects were tested in order to make sure that everything works after migration:</summary>

### Example apps

- [x] basic-example
  - [x] Android
  - [x] iOS
- [x] expo-example (fabric)
  - [x] Android
  - [x] iOS
  - [x] web (in that case architecture doesn't matter)
- [x] expo-example (paper)
  - [x] Android
  - [x] iOS
- [x] macos-example

### Scripts

- [x] check-android-dirs
- [x] check-rn-versions
- [x] format-android
- [x] format-apple
- [x] codegen-check-consistency
- [x] codegen-sync-archs
- [x] clangd

### Linters

- [x] prettier
- [x] eslint
- [x] lint-staged

### docs

- [x] check documentation

### Build from generated package

- [x] New app builds correctly

</details>
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