Skip to content

Enable jdk20 in ci#736

Merged
bodiam merged 1 commit into
datafaker-net:mainfrom
snuyanzin:jdk20
Mar 25, 2023
Merged

Enable jdk20 in ci#736
bodiam merged 1 commit into
datafaker-net:mainfrom
snuyanzin:jdk20

Conversation

@snuyanzin

Copy link
Copy Markdown
Collaborator

No description provided.

@what-the-diff

what-the-diff Bot commented Mar 25, 2023

Copy link
Copy Markdown

PR Summary

  • Java Version Update - Changed Java version from 19 to 20
  • Enhanced Build Compatibility - Added windows-latest as an option for the build job

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #736 (d870e49) into main (75f9837) will decrease coverage by 0.02%.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@             Coverage Diff              @@
##               main     #736      +/-   ##
============================================
- Coverage     92.90%   92.88%   -0.02%     
+ Complexity     2585     2584       -1     
============================================
  Files           281      281              
  Lines          5114     5114              
  Branches        528      528              
============================================
- Hits           4751     4750       -1     
  Misses          235      235              
- Partials        128      129       +1     

see 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@bodiam bodiam merged commit da3d4c6 into datafaker-net:main Mar 25, 2023
@kingthorin

Copy link
Copy Markdown
Collaborator

I was waiting for qodana to finish but maybe it’s got issues 🤷‍♂️

@bodiam

bodiam commented Mar 25, 2023

Copy link
Copy Markdown
Contributor

I think it's not correctly configured, not sure. Last time I checked the data was out of date, but I don't think the jvm version should affect this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants