-
Notifications
You must be signed in to change notification settings - Fork 26
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
Add support for Gradle v7, fixes #164 #168
Conversation
|try { | ||
| java.nio.file.Files.write( | ||
| java.nio.file.Paths.get('$gradleVersionPath'), | ||
| [gradle.gradleVersion], |
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.
Is this some scala feature Im not aware of?
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.
Which feature? This is a Groovy script, where you have literal syntax [a]
for lists.
case Some(gradleVersion) | ||
if gradleVersion.startsWith("6.7") && | ||
toolchains.toolchains.nonEmpty => | ||
index | ||
.app | ||
.error( | ||
"lsif-java does not support Gradle 6.7 when used together with Java toolchains. " + | ||
"To fix this problem, upgrade to Gradle version 6.8 or newer and try again." | ||
) | ||
CommandResult(1, Nil) | ||
case _ => | ||
runCompileCommand(toolchains) |
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.
Will this still call runCompileCommand
if if gradleVersion.startsWith("6.7") && toolchains.toolchains.nonEmpty
evaluates to false?
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.
Yes, case _ =>
is a catchall
Also adds a custom error message for users on Gradle v6.7 using the experimental toolchains feature. We can only support toolchains from version v6.8.
Also adds a custom error message for users on Gradle v6.7 using the
experimental toolchains feature. We can only support toolchains from
version v6.8.