-
Notifications
You must be signed in to change notification settings - Fork 923
Java21 #7784
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
Java21 #7784
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7784 +/- ##
=========================================
Coverage 90.17% 90.18%
- Complexity 7190 7191 +1
=========================================
Files 814 814
Lines 21724 21724
Branches 2129 2129
=========================================
+ Hits 19589 19591 +2
+ Misses 1467 1466 -1
+ Partials 668 667 -1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| "-Xlint:-serial", | ||
| "-Xlint:-this-escape", |
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.
👀
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.
Resolved in #7785
| // target. | ||
| java { | ||
| toolchain { | ||
| languageVersion.set(JavaLanguageVersion.of(17)) |
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 it no longer relevant to set the java language version for the build of buildSrc?
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.
I don't think so, other repos don't
Currently built on top of #7780Motivated by #7779