Skip to content

chore: bump react-native to 0.66 #757

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

Merged
merged 2 commits into from
Mar 15, 2022
Merged

chore: bump react-native to 0.66 #757

merged 2 commits into from
Mar 15, 2022

Conversation

tido64
Copy link
Member

@tido64 tido64 commented Feb 18, 2022

Summary

Bump react-native to 0.66

Test Plan

CI should pass.

@tido64 tido64 marked this pull request as ready for review February 18, 2022 14:25
@tido64 tido64 requested a review from krizzu February 18, 2022 14:25
@tido64
Copy link
Member Author

tido64 commented Mar 3, 2022

@krizzu: Can you review this whenever you have some free cycles? Thanks 😄

};
project: {
android: {
sourceDir: path.join('example', 'android'),
Copy link
Member

Choose a reason for hiding this comment

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

q: does this allow us to remove root option from run-android script?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, it does.

),
),
},
ios: {
Copy link
Member

Choose a reason for hiding this comment

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

Because we specify options in start:macos script, we don't have to add this target here, right?

Copy link
Member Author

Choose a reason for hiding this comment

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

react-native-macos doesn't use this field, I think. They look for the project independent of the cli config.

Comment on lines 62 to 65
packageSatisfiesVersionRange(
'@react-native-community/cli-platform-ios',
'<5.0.2',
)
Copy link
Member

Choose a reason for hiding this comment

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

👏 nice

Copy link
Member

@krizzu krizzu left a comment

Choose a reason for hiding this comment

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

Well done sir 👏 . There's higher version of test app on master (hence conflict)

@tido64 tido64 force-pushed the tido/bump-react-native branch from 3810aeb to 7fe2b59 Compare March 15, 2022 13:11
@tido64 tido64 force-pushed the tido/bump-react-native branch from 7fe2b59 to 2eace8e Compare March 15, 2022 13:24
@tido64 tido64 merged commit ecda1c0 into master Mar 15, 2022
@tido64 tido64 deleted the tido/bump-react-native branch March 15, 2022 13:55
@krizzu
Copy link
Member

krizzu commented Mar 16, 2022

🎉 This PR is included in version 1.16.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

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