-
Notifications
You must be signed in to change notification settings - Fork 96
[eclipse/xtext-core#829] Update to Gradle Wrapper 4.9 #830
Conversation
Signed-off-by: Christian Dietrich <christian.dietrich@itemis.de>
Dont merge yet CQ is still outstanding |
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.
Wait for CQ approval
distributionBase=GRADLE_USER_HOME | ||
distributionPath=wrapper/dists | ||
zipStoreBase=GRADLE_USER_HOME | ||
zipStorePath=wrapper/dists | ||
distributionUrl=https\://services.gradle.org/distributions/gradle-4.6-bin.zip | ||
distributionUrl=https\://services.gradle.org/distributions/gradle-4.9-bin.zip |
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.
Should we use gradle-4.9-all.zip instead to sport debugging of the build?
am not sure. i just a |
We would need to set a different distribution type in the wrapper task. Not sure it's worth it though: |
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.
CQ is approved, go!
[#829] Update to Gradle Wrapper 4.9
Signed-off-by: Christian Dietrich christian.dietrich@itemis.de