search

belgif / rest-guide-validator

A validator that checks compliance of OpenAPI documents with the Belgif REST guide
Apache License 2.0
5 stars 0 forks source link

Upgrade to JDK 17 #9

Closed pvdbosch closed 3 weeks ago

pvdbosch commented 3 weeks ago

Analyze if we can update to JDK 17. This requires JDK >= 17 when running Maven (usually determined by JDK_HOME env var), but compilation of project code using the plugin can still be done by an earlier JDK version in the pom configuration.

@jpraet : is CBSS okay with updating the JDK requirement for the plugin? Still to check for other large users.

Implementation of update already in https://github.com/belgif/rest-guide-validator/pull/8

jpraet commented 3 weeks ago

Yes, CBSS uses JDK 21 to run Maven on workstations and jenkins.

pvdbosch commented 3 weeks ago

we'll go ahead with upgrading to JDK 17. We should add a 'Prerequisites' section to the readme.md to specify minimum JDK and maven version. With clarification that JDK version is for maven runtime, not for source code compilation. IDEs might need to be manually configured for maven runtime JDK version.