Skip to content
This repository has been archived by the owner on Nov 30, 2023. It is now read-only.

[java] Fix symlink and upgrade to Gradle 5.4.1 #30

Merged
merged 4 commits into from May 7, 2019

Conversation

brunoborges
Copy link
Member

This PR fixes the wrong symlink to the Gradle binary and upgrades to version 5.4.1

@Chuxel Chuxel requested a review from testforstephen May 4, 2019 22:10
Copy link
Collaborator

@testforstephen testforstephen left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@Chuxel Chuxel requested a review from chrmarti May 6, 2019 14:44
@Chuxel Chuxel added the java label May 6, 2019
@Chuxel
Copy link
Member

Chuxel commented May 6, 2019

@chrmarti I think this is safe to merge at this point - do you still want to hold off given its a bug fix?

@Chuxel
Copy link
Member

Chuxel commented May 6, 2019

@brunoborges @testforstephen @chrmarti Actually I just retried and the SHA check failed. Do we need that given this will be a moving target?

@brunoborges
Copy link
Member Author

@Chuxel I've updated the hashes. Pls try again?

@Chuxel
Copy link
Member

Chuxel commented May 7, 2019

LGTM! @chrmarti - LMK if you want to merge the fix.

@chrmarti
Copy link
Contributor

chrmarti commented May 7, 2019

Bug fixes are good to merge. We will follow the same process as with VS Code at this point. That includes a review (can be yourself if the PR is from the community) and verification of the fixes before we publish a new version (we can download the vsix from the build).

@Chuxel Chuxel merged commit 39c80bf into microsoft:master May 7, 2019
@brunoborges brunoborges deleted the fix-gradle-link branch May 8, 2019 03:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants