feat: bumped to java 11, altered CI to target java 11

[jarebudev]

This PR

Moves the sdk to java 11

Related Issues

Resolves #1371

Notes

Follow-up Tasks

If desired some follow on work could be to do some minor refactoring to use JDK 11 language features.

How to test

N/A

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.25%. Comparing base (a6389e8) to head (07ef230).

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              

Flag	Coverage Δ
unittests	93.25% <ø> (+0.44%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[aepfli]

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. :)

[jarebudev]

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.

[aepfli]

from my perspective this looks ready to get review, if you think you are ready, please let me know, than I can approve, and we can get this merged :)

[jarebudev]

@aepfli yep it's good for review, thanks!

[aepfli]

hey, there sadly is a merge conflict, which would be cool if you could resolve it.

Furthermore your work inspired me and I created https://github.com/open-feature/java-sdk/tree/jarebudev-1371_java11_refactor - based on your branch. Adding a module info file, which would be need to have - but I am not sure, if we should split it into a separate pr :) wdyt?

... and before we merge that, we should most likely update our sdk-contrib repo also to use a newer java version. If you would be willing to contribute the same changes to https://github.com/open-feature/java-sdk-contrib/ this would be highly appreciated, else i am more than glad to apply your changes also to the contrib repo :)

[jarebudev]

hey, there sadly is a merge conflict, which would be cool if you could resolve it.

Merge conflict is now resolved.

Furthermore your work inspired me and I created https://github.com/open-feature/java-sdk/tree/jarebudev-1371_java11_refactor - based on your branch. Adding a module info file, which would be need to have - but I am not sure, if we should split it into a separate pr :) wdyt?

Would be nice to have a module-info, it's a small addition to this PR. I'm happy to include it.

... and before we merge that, we should most likely update our sdk-contrib repo also to use a newer java version. If you would be willing to contribute the same changes to https://github.com/open-feature/java-sdk-contrib/ this would be highly appreciated, else i am more than glad to apply your changes also to the contrib repo :)

Yeh I can update the sdk-contrib, will make the changes and put up a MR.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud