Skip to content

[Monorepo] Move bob build to build script#3525

Merged
m-bert merged 2 commits into
@mbert/monorepofrom
@mbert/monorepo-move-bob
May 23, 2025
Merged

[Monorepo] Move bob build to build script#3525
m-bert merged 2 commits into
@mbert/monorepofrom
@mbert/monorepo-move-bob

Conversation

@m-bert
Copy link
Copy Markdown
Collaborator

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

Description

When I tried to install GestureHandler from newly created package, it fails with message:

command not found: bob

This PR moves bob build into build script (as it is already done in Reanimated), so that postinstall no longer fails.

Test plan

Generate new package and add it to newly created app.

@m-bert m-bert changed the base branch from main to @mbert/monorepo May 22, 2025 15:46
@m-bert m-bert requested a review from j-piasecki May 22, 2025 15:47
@m-bert m-bert marked this pull request as draft May 22, 2025 15:48
@m-bert m-bert marked this pull request as ready for review May 22, 2025 15:59
@m-bert m-bert mentioned this pull request May 22, 2025
23 tasks
@m-bert m-bert merged commit 919177f into @mbert/monorepo May 23, 2025
2 checks passed
@m-bert m-bert deleted the @mbert/monorepo-move-bob branch May 23, 2025 06:21
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