-
Notifications
You must be signed in to change notification settings - Fork 458
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
Upgrade to JDK 21 #2410
Upgrade to JDK 21 #2410
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2410 +/- ##
============================================
- Coverage 66.88% 66.88% -0.01%
Complexity 1410 1410
============================================
Files 208 208
Lines 12110 12112 +2
Branches 1487 1488 +1
============================================
+ Hits 8100 8101 +1
Misses 3494 3494
- Partials 516 517 +1 ☔ View full report in Codecov by Sentry. |
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 with a question
@@ -224,7 +236,7 @@ | |||
</transformers> | |||
<!-- This fixes the issue "Invalid signature file digest for Manifest main attributes" | |||
cf. http://zhentao-li.blogspot.com/2012/06/maven-shade-plugin-invalid-signature.html --> | |||
<filters> | |||
<!--filters> |
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 change intentional?
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, commented out intentionally - not sure its effect... so punting for now, will circle back later to figure out implications.
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
No description provided.