-
Notifications
You must be signed in to change notification settings - Fork 46
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
Java 17 features in code #882
Conversation
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.
Thanks for the PR! You'll need to revert the changes which convert anonymous inner classes to lambdas. It's a limitation of Gradle's configuration cache: lambdas need to be Serializable
.
.collect(Collectors.joining("\n ")) | ||
); | ||
} | ||
task.doFirst((t) -> { |
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.
As far as I'm aware, you'll need to revert this change: lambdas are not compatible with the configuration cache.
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.
fixed
if (lines.stream().noneMatch(l -> l.contains("Snapshotting complete"))) { | ||
throw new GradleException("Checkpoint container failed"); | ||
} | ||
task.doLast(t -> { |
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.
Same here, we need to use anonymous classes I think.
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.
fixed
t.getLogger().warn(" CRaC checkpoint files may contain sensitive information."); | ||
t.getLogger().warn("**********************************************************"); | ||
} | ||
task.doLast(t -> { |
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.
Same here.
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.
fixed
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.
Many thanks!
You're welcome! ;-) |
Fixed tests for windows