Skip to content

[Monorepo] Move scripts directory to root#3506

Merged
m-bert merged 79 commits into@mbert/monorepofrom
@mbert/monorepo-move-scripts
May 12, 2025
Merged

[Monorepo] Move scripts directory to root#3506
m-bert merged 79 commits into@mbert/monorepofrom
@mbert/monorepo-move-scripts

Conversation

@m-bert
Copy link
Copy Markdown
Collaborator

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

Description

This PR moves scripts directory to root of the monorepo. It also

  • Brings back clangd support,
  • Renames some of scripts to be global

Test plan

Check that CI passes and that building iOS app generates compile_commands.json file.

@m-bert m-bert mentioned this pull request May 8, 2025
23 tasks
@m-bert m-bert requested a review from j-piasecki May 8, 2025 15:37
Base automatically changed from @mbert/monorepo-unify-eslint-configs to @mbert/monorepo May 9, 2025 08:10
@m-bert m-bert marked this pull request as ready for review May 9, 2025 08:12
Comment thread packages/react-native-gesture-handler/package.json Outdated
@m-bert m-bert requested a review from j-piasecki May 12, 2025 10:14
@m-bert m-bert merged commit 5140914 into @mbert/monorepo May 12, 2025
9 checks passed
@m-bert m-bert deleted the @mbert/monorepo-move-scripts branch May 12, 2025 13:50
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