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 Gradle to 5.4.1 #24542

Closed
wants to merge 3 commits into from
Closed

Conversation

dulmandakh
Copy link
Contributor

@dulmandakh dulmandakh commented Apr 20, 2019

Summary

Bump Gradle to 5.4.1, includes many improvements and fixes.

Changelog

[Android] [Changed] - Bump Gradle to 5.4.1

Test Plan

RNTester is running as expected.

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Contributor A React Native contributor. Partner labels Apr 20, 2019
@dulmandakh
Copy link
Contributor Author

@gengjiawen @hey99xx what is you opinion about JVM options changes?

@gengjiawen
Copy link
Contributor

why change the jvm options ?

@dulmandakh
Copy link
Contributor Author

dulmandakh commented Apr 20, 2019

./gradlew wrapper task did change it

@gengjiawen
Copy link
Contributor

./gradlew wrapper task did change it

I tried the change locally, works as expected.
But I didn't find related changelog. Anyway, I think this is good to ship.

@dulmandakh dulmandakh changed the title bump Gradle to 5.4 bump Gradle to 5.4.1 Apr 26, 2019
@hramos hramos self-assigned this Apr 26, 2019
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.

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

@react-native-bot
Copy link
Collaborator

This pull request was successfully merged by @dulmandakh in 6976a93.

When will my fix make it into a release? | Upcoming Releases

@react-native-bot react-native-bot added the Merged This PR has been merged. label Apr 27, 2019
kelset pushed a commit that referenced this pull request Jun 5, 2019
Summary:
Bump Gradle to 5.4.1, includes many improvements and fixes.

[Android] [Changed] - Bump Gradle to 5.4.1
Pull Request resolved: #24542

Differential Revision: D15107106

Pulled By: hramos

fbshipit-source-id: aeae0413a720466f1eef00ad8e80d59c406ecd80

# Conflicts:
#	template/android/gradle/wrapper/gradle-wrapper.jar
douglowder pushed a commit to react-native-tvos/react-native-tvos that referenced this pull request Sep 11, 2019
Summary:
Bump Gradle to 5.4.1, includes many improvements and fixes.

[Android] [Changed] - Bump Gradle to 5.4.1
Pull Request resolved: facebook/react-native#24542

Differential Revision: D15107106

Pulled By: hramos

fbshipit-source-id: aeae0413a720466f1eef00ad8e80d59c406ecd80

# Conflicts:
#	template/android/gradle/wrapper/gradle-wrapper.jar
@dulmandakh dulmandakh deleted the gradle-5.4 branch October 15, 2019 05:14
This pull request was closed.
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. Contributor A React Native contributor. Merged This PR has been merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants