-
Notifications
You must be signed in to change notification settings - Fork 24.4k
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
bump targetSdkVersion to 28 #23431
bump targetSdkVersion to 28 #23431
Conversation
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.
This is gonna be in 0.60, is that your intention?
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.
@cpojer is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
@cpojer maybe we can cherry pick it to 0.59, because Android Studio is highlighting targetSdkVersion and suggesting to use the latest one. But I don't have opinion about it. |
@dulmandakh merged commit 57f444b into |
Summary: Bump targetSdkVersion to 28 [Android] [Changed] - Bump targetSdkVersion to 28 Pull Request resolved: #23431 Differential Revision: D14065177 Pulled By: cpojer fbshipit-source-id: a161d1d385b7b40ec93d70851e5a41baeb58f830
Summary: Bump targetSdkVersion to 28 [Android] [Changed] - Bump targetSdkVersion to 28 Pull Request resolved: facebook/react-native#23431 Differential Revision: D14065177 Pulled By: cpojer fbshipit-source-id: a161d1d385b7b40ec93d70851e5a41baeb58f830
Summary
Bump targetSdkVersion to 28
Changelog
[Android] [Changed] - Bump targetSdkVersion to 28
Test Plan
CI is green, and apps will work as usual.