Skip to content
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 Android SDK to 28 #21632

Closed
wants to merge 6 commits into from
Closed

Conversation

dulmandakh
Copy link
Contributor

@dulmandakh dulmandakh commented Oct 10, 2018

This PR is bumping compileSdkVersion to 28, buildToolsVersion to 28.0.3, Gradle to 4.10.2, Android Gradle plugin to 3.2.1.

Gradle 4.7 added support for Java 10 and 11.

Test Plan:

CI is green, and RNTester app is working as expected.

Release Notes:

[ANDROID] [ENHANCEMENT] [SDK] - bump to 28

@dulmandakh dulmandakh requested a review from hramos October 10, 2018 06:55
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Oct 10, 2018
@dulmandakh
Copy link
Contributor Author

bumped Gradle to 4.10.2 because gradle-download-task landed tests for it. Also bumped android plugin to 3.2.1.

@dulmandakh
Copy link
Contributor Author

@hramos bumped tools, but keeping target SDK.

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@mdvacca has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@hramos
Copy link
Contributor

hramos commented Dec 1, 2018

This was blocked for a while, as upgrading the Gradle binary internally takes some manual work.

The problem I'm running into now is that while Android SDK 28 is available in our internal repository, we do not yet have access to Build Tools 28.0.3. The latest version we have is 28.0.2, but Gradle 4.10.2's minimum supported version is 28.0.3.

@hramos
Copy link
Contributor

hramos commented Dec 5, 2018

This is landing now. As build tools 28.0.3 are not available on our internal build servers yet, I had to downgrade the gradle plugin as 3.2.1 requires 28.0.3. The combination I landed on is compileSdkVersion to 28, buildToolsVersion to 28.0.2, Gradle to 4.7, and Android Gradle plugin to 3.2.0.

@react-native-bot
Copy link
Collaborator

@dulmandakh merged commit 9d00d4d into facebook:master.

@facebook facebook locked as resolved and limited conversation to collaborators Dec 5, 2018
@react-native-bot react-native-bot added the Merged This PR has been merged. label Dec 5, 2018
grabbou pushed a commit that referenced this pull request Dec 17, 2018
…Plugin to 3.2.0 (#21632)

Summary:
This PR is bumping compileSdkVersion to 28, buildToolsVersion to 28.0.2, Gradle to 4.7, Android Gradle plugin to 3.2.0.

Gradle 4.7 added support for Java 10 and 11.

allow-large-files

Release Notes:
--------------
[ANDROID] [ENHANCEMENT] [SDK] - bump to 28

Pull Request resolved: #21632

Reviewed By: mdvacca

Differential Revision: D13084836

Pulled By: hramos

fbshipit-source-id: e0f493881e80e87faf8c3ef1ac77044495966a49
@hramos
Copy link
Contributor

hramos commented Jan 11, 2019

Build tools 28.0.3 are now available internally. I have a diff out that will update us to 28.0.3, as well as Gradle 4.10.2 and Gradle Plugin 3.2.1.

@dulmandakh
Copy link
Contributor Author

dulmandakh commented Jan 11, 2019

f3e5cce bumped gradle to 4.10.2 and remove build tools version. Now only need to bump gradle android plugin.

@hramos
Copy link
Contributor

hramos commented Jan 11, 2019

That commit bumped the Gradle version used in OSS to 4.10.2, but our internal Gradle tests were still on 4.7 and therefore failing internally.

@dulmandakh dulmandakh deleted the bump-sdk-28 branch February 22, 2019 14:23
t-nanava pushed a commit to microsoft/react-native-macos that referenced this pull request Jun 17, 2019
…Plugin to 3.2.0 (facebook#21632)

Summary:
This PR is bumping compileSdkVersion to 28, buildToolsVersion to 28.0.2, Gradle to 4.7, Android Gradle plugin to 3.2.0.

Gradle 4.7 added support for Java 10 and 11.

allow-large-files

Release Notes:
--------------
[ANDROID] [ENHANCEMENT] [SDK] - bump to 28

Pull Request resolved: facebook#21632

Reviewed By: mdvacca

Differential Revision: D13084836

Pulled By: hramos

fbshipit-source-id: e0f493881e80e87faf8c3ef1ac77044495966a49
rozele pushed a commit to microsoft/react-native-windows that referenced this pull request Jul 18, 2019
…Plugin to 3.2.0 (#21632)

Summary:
This PR is bumping compileSdkVersion to 28, buildToolsVersion to 28.0.2, Gradle to 4.7, Android Gradle plugin to 3.2.0.

Gradle 4.7 added support for Java 10 and 11.

allow-large-files

Release Notes:
--------------
[ANDROID] [ENHANCEMENT] [SDK] - bump to 28

Pull Request resolved: facebook/react-native#21632

Reviewed By: mdvacca

Differential Revision: D13084836

Pulled By: hramos

fbshipit-source-id: e0f493881e80e87faf8c3ef1ac77044495966a49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. Platform: Android Android applications.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants