chore: move to monorepo with bun#4255
chore: move to monorepo with bun#4255KrzysztofMoch wants to merge 27 commits intoTheWidlarzGroup:support/6.x.xfrom
Conversation
10ef3a3 to
e64d810
Compare
|
@freeboub this PR will create conflicts in all PRs, so I would like to merge as many PRs as possible before this one |
|
Also builds are faster by |
|
finally a clean repo 🙏 Thank you ! |
|
@KrzysztofMoch I would be happy if this PR is merged soon! |
844de46 to
c36d69f
Compare
|
@freeboub I fine to merge it - other PRs will need to rebase but I guess this is probably inevitable |
c36d69f to
8f629f1
Compare
|
@KrzysztofMoch I am not able to build samples with this PR ... on bare sample: info 💡 Tip: Make sure that you have set up your development environment correctly, by running npx react-native doctor. To read more about doctor command visit: https://github.com/react-native-community/cli/blob/main/packages/cli-doctor/README.md#doctor FAILURE: Build failed with an exception.
SyntaxError: Unexpected token 'typeof' Node.js v22.9.0
BUILD FAILED in 466ms
I also think you should move react-native-video-plugin-sample to exemple folder as it is not made to be released. Thank you |
|
pod install is also failing on ios: SyntaxError: Unexpected token 'typeof' Node.js v22.9.0 Failed to apply config plugins /Users/olivier/openSource/react-native-video/examples/bare/node_modules/react-native-test-app/ios/test_app.rb:29:in [!] CocoaPods will not include resources it cannot find: The app will still build and run if they are served by the dev server. To include missing resources, make sure they exist, then run |
|
I have bumped react-native version in my versions
|
ec43ea6 to
1c06c15
Compare
|
@freeboub it took a while but it's should be good now |
|
@KrzysztofMoch sorry for the testing delays, but it is still not working on my side... I started from my existing repo , can it be the issue ? on bare project: info 💡 Tip: Make sure that you have set up your development environment correctly, by running npx react-native doctor. To read more about doctor command visit: https://github.com/react-native-community/cli/blob/main/packages/cli-doctor/README.md#doctor FAILURE: Build failed with an exception.
SyntaxError: Unexpected token 'typeof' Node.js v23.1.0
BUILD FAILED in 438ms
in expo project: FAILURE: Build failed with an exception.
BUILD FAILED in 1s |
|
Hmm... We have my favourite situation "it's working on my machine" 🥲 |
It was expecting this answer 😂 I will try a clean clone to check if it fixes the issue... Maybe a node version issue ? I don't know ... |
And we will not ship your computer to customers 😁 |
|
What version of |
c960201 to
3773a19
Compare
|
@KrzysztofMoch Still not, but great progress ! I think you just miss to add the splash screen file (expo exemple) Execution failed for task ':app:processDebugResources'.
|
|
oh, i think i forgot to commit new prebuild, can you copy settings.gradle and make new prebuild ? |
Seems like RNTA does not work yet with 0.76.5
d2f2523 to
38e4b0c
Compare
|
@freeboub - I have fixed issue with missing splash screen and not loading app (in expo), if this won't work I give up |
|
@KrzysztofMoch I finally find time to test it, and, good news, it works !!! |
|
I will look into other PRs and merge this by end of week! |
|
Any updates on this? |
Summary
I have replaced yarn with bun as it is much faster.
Also changed our repo structure to mono repo so we have now:
-> docs
-> examples
| -> bare
| -> expo
-> packages
| -> react-native-video (library code)
| -> react-native-video-plugin-sample
Motivation
Speed up "node_modules" installation and allow for installing react-native-video as dependency in examples with simplify configuration.
Changes
Test plan