Repository navigation
Update gradle version to 4.4.1. - #186
Conversation
| task wrapper(type: Wrapper) { | ||
| gradleVersion = '4.4.1' | ||
| distributionUrl = "https://services.gradle.org/distributions/gradle-$gradleVersion-all.zip" | ||
| } |
There was a problem hiding this comment.
Please remove this task. You can do this without adding a task to build.gradle:
gradlew wrapper --gradle-version 4.4.1
|
Thanks for the PR! As mentioned in the issue, we want to keep compat with |
|
Edit: apologies - you already used Testkit to fix to 2.14.1! Thanks! |
|
Once CI is fixed and the wrapper task is removed, this LGTM! |
f8ff66a to
51ab5e1
Compare
| @@ -1,2 +1,2 @@ | |||
| * text eol=lf | |||
| * text=auto | |||
There was a problem hiding this comment.
for some reason eol=lf was doing something weird to gradle/wrapper/gradle-wrapper.jar and was causing it not to be usable. this caused the previous CI failure. changing this to text=auto fixed it.
There was a problem hiding this comment.
This change will break the tests when run on a windows machine. Maybe instead add a line marking gradle-wrapper.jar as binary?
There was a problem hiding this comment.
From the documentation it seems that auto should automatically set the line endings to LF when git detects that it is a text file. However, I don't have a windows machine to test this one. Do you?
Set to string value "auto"
When text is set to "auto", the path is marked for automatic end-of-line conversion. If Git decides that the content is text, its line endings are converted to LF on checkin. When the file has been committed with CRLF, no conversion is done.
From: https://www.kernel.org/pub/software/scm/git/docs/gitattributes.html
There was a problem hiding this comment.
Looks like you're right, but I'd still rather not depend on Git's auto-detection.
Right now it's this, which has worked for a long time:
* text eol=lf
*.png binary
This seems like the safest bet:
* text eol=lf
*.png binary
*.jar binary
There was a problem hiding this comment.
Ok, i'll change it then.
Also maintain testing compatibility with 2.14.1. Fixes diffplug#161
51ab5e1 to
569650d
Compare
|
Updated to mark |
|
Great, thanks @runningcode! |
2 participants
Also maintain testing compatibility with 2.14.1.
Fixes #161