-
Notifications
You must be signed in to change notification settings - Fork 4.5k
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
Update Groovy to 3.0.10 #20177
Update Groovy to 3.0.10 #20177
Conversation
de98d57
to
29702f4
Compare
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.
LGTM
Waiting on javaparser/javaparser#3516. |
@octylFractal could you see if |
|
Hrm, there probably should have been a 3.0.x at some point. Don't worry about it now |
Nice!! |
HOw is that issue blocking this? Aren't all checks passed here? |
It's a security concern, related to #20177 (comment) -- I am unsure if the change is intentional, and if it wasn't, then it could be a sign of malicious code. Obviously we prefer not to distribute malicious code :) |
Thanks!! Sounds great!! |
I think nothing's blocking this now, right? 😉😉😉😉😉 |
@bot-gradle test and merge |
Error while scheduling pre-tested commit, please contact the administrator. |
Required for JDK 18 support.
7625fc3
to
6540c81
Compare
@bot-gradle test and merge |
OK, I've already triggered a build for you. |
Noice |
Required for JDK 18 support. Partial work on #19283.