Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[js/rn] Supoort New Architecture #16669
base: main
Are you sure you want to change the base?
[js/rn] Supoort New Architecture #16669
Changes from 42 commits
2354db1
209b91f
efce8ed
f176211
70daa8d
39dab48
8485286
1df0795
16cff1e
cff8ff2
e6f5bff
d99af7f
e0813d3
0ce934a
b12e78d
a2f98f6
7dde3a6
e7a1c9d
c9e1714
61ff7ab
e4e389b
1d55dc4
77f54a3
f964488
ca38c81
5ba46e3
200f14b
7ea8b98
27d36df
b02c8e2
3babf7e
075770f
9479f46
99817d6
4560820
b50162a
ba5dc5d
0b6479e
8599948
4fd5831
93c195b
110b381
48e0e73
0a1988a
5e08a9e
e1a5b8d
446b12b
c409719
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
general:
I guess you mentioned that we are still using RN v0.69 for the unit tests, does it cause issue if we are using inconsistent react native versions across the project? Asking as we would really want to enable some tests to cover this new architecture, so wondering if it can cause issue when we running that in CI.
same question for the E2E test project, is it available for testing the new turbo module with the current set up?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Previously I was not able run the
react_native/android
unit tests successfully with 0.71.x.Today I tried again, it should be easy by use
react-android
for >= 0.71:For RN project, the
react-native-gradle-plugin
should always convertedreact-native
toreact-android
, but it's not used in the unit tests.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Got it. that's great!