Skip to content

Conversation

@friederbluemle
Copy link
Contributor

@friederbluemle friederbluemle commented Feb 8, 2019

Summary

Projects generated with the latest stable version of Android Studio come with Gradle wrapper 4.10.1. The Gradle documentation states:

NOTE: Gradle 4.10 has had three patch releases. You should use the latest patch release (4.10.3).

Currently, react-native is using 4.10.2. This PR brings Gradle wrapper from 4.10.2 to 4.10.3. Updates were automatically performed using the gradle wrapper task.

Related: #23324

Changelog

[Android] [Added] - Update Gradle wrapper to 4.10.3

Test Plan

Regular testing. No changes.

@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 Feb 8, 2019
@friederbluemle friederbluemle force-pushed the update-gradle-wrapper branch 2 times, most recently from 1b6a901 to 52814ed Compare February 10, 2019 06:23
@friederbluemle
Copy link
Contributor Author

Rebased onto latest master.

@friederbluemle friederbluemle force-pushed the update-gradle-wrapper branch 2 times, most recently from 48a4b98 to 313548e Compare February 11, 2019 07:29
@hramos
Copy link
Contributor

hramos commented Feb 11, 2019

We're updating to Gradle 5.0 in #23324.

@friederbluemle
Copy link
Contributor Author

@hramos - Thanks for your comment. Yes, I know - That's why I linked the PR in the description :) As much as I applaud the usage of newer versions, don't you think it would be advisable to keep the Gradle wrapper version in sync with whatever the currently used Android Gradle plugin officially supports? For Android Gradle plugin 3.3.0 and 3.3.1 (the latest) that is Gradle 4.10.x.

@hramos
Copy link
Contributor

hramos commented Feb 12, 2019

I'll let you and @dulmandakh sort this out.

@hramos
Copy link
Contributor

hramos commented Feb 13, 2019

I'm going to close this, as #23324 is landing. Happy to hear any arguments in favor of downgrading to 4.10.3 from 5.0. I'm following @dulmandakh advice given he has been actively sending PRs to upgrade our Gradle config over the past year.

@hramos hramos closed this Feb 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants