Skip to content
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

Change baseline to require Java 17 minimum #12415

Merged
merged 5 commits into from Jan 14, 2024

Conversation

chadlwilson
Copy link
Member

@chadlwilson chadlwilson commented Dec 30, 2023

  • Increases the baseline Java version to 17, which has been out for 2.5 years now, and has been packaged with GoCD for quite a long time now
  • Fixes the go-plugin-api version at Java 11 to avoid plugins having to increase their own version just to stay compatible with newer versions of the plugin API
  • Fixes issues with JVM version of the jar-class-loader which is intended to target old versions to be able to display useful errors to users. This was intended to support Java 8 onwards, but has been broken for a while and was actually effectively Java 11 on 23.5.0 and back a number of versions. This fixes that.
  • Removes compatibility declarations from MANIFEST.MF as some where inaccurate, and it is not a mandatory declaration. It's also misleading in some cases of uber/fatjars. Seems unnecessary.

@chadlwilson chadlwilson added technical java Pull requests that update Java code labels Dec 30, 2023
@chadlwilson chadlwilson added this to the Release 24.1.0 milestone Dec 30, 2023
@chadlwilson chadlwilson force-pushed the java-17-minimum branch 4 times, most recently from c29f57c to 1c31e84 Compare January 5, 2024 04:21
@chadlwilson chadlwilson force-pushed the java-17-minimum branch 11 times, most recently from 0329ad7 to ed37d16 Compare January 14, 2024 08:59
@chadlwilson chadlwilson marked this pull request as ready for review January 14, 2024 09:45
@chadlwilson chadlwilson merged commit 0440ffe into gocd:master Jan 14, 2024
1 of 2 checks passed
@chadlwilson chadlwilson deleted the java-17-minimum branch January 14, 2024 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
java Pull requests that update Java code technical
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant