-
Notifications
You must be signed in to change notification settings - Fork 42
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
feat: bumped to java 11, altered CI to target java 11 #1393
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: jarebudev <[email protected]>
Signed-off-by: jarebudev <[email protected]>
Signed-off-by: jarebudev <[email protected]>
Signed-off-by: jarebudev <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1393 +/- ##
============================================
+ Coverage 92.81% 93.25% +0.44%
- Complexity 469 471 +2
============================================
Files 43 43
Lines 1127 1127
Branches 91 91
============================================
+ Hits 1046 1051 +5
+ Misses 53 48 -5
Partials 28 28
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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.
Hey, the changes to the github action and pom look great, thank you. This is highly appreciated, but should we really change all the java code, it just adds unnecessary changes to the pull request. :)
No worries, I can revert the last commit and leave the source unchanged if that's desired. Can always reapply JDK11 language refactoring afterwards in another PR. |
678e04e
to
53fbb2c
Compare
Signed-off-by: jarebudev <[email protected]>
|
draft PR - have added it as wanted to verify my initial changes builds the CI first up before going further
This PR
Moves the sdk to java 11
Related Issues
Resolves #1371
Notes
draft at the moment
Follow-up Tasks
N/A
How to test
N/A