-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Bump JDK 21 from ea to GA #2278
Conversation
JDK 21 is go GA now
Codecov Report
@@ Coverage Diff @@
## master #2278 +/- ##
============================================
+ Coverage 85.11% 85.14% +0.02%
+ Complexity 1239 1238 -1
============================================
Files 215 215
Lines 3984 3984
Branches 454 454
============================================
+ Hits 3391 3392 +1
Misses 386 386
+ Partials 207 206 -1 see 1 file with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. 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.
- CI is fixed in the master branch, please merge the master branch.
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 tweaked some of the unit tests that affected Windows CI at Relax the conditions for unit tests involving awaitility to fix CI #2293 .
-
LGTM!
Changes proposed in this pull request: