jakartaee / rest

Jakarta RESTful Web Services
Other
351 stars 114 forks source link

Add the enforcer plugin and require a minimum of Java 21 to compile t… #1203

Closed jamezp closed 5 months ago

jamezp commented 6 months ago

The minimum level should remain at Java SE 17. The TCK itself should also require Java SE 17 as a minimum.

resolves #1202

jamezp commented 6 months ago

Please note I've changed this PR to have the TCK compile to Java SE 17 as well. It was noted in the Jakarta EE spec meeting there was never an intent for specification TCK's to require Java 21.

jansupol commented 6 months ago

So we can just set release to 17 in the compiler plugin and drop the enforcer plugin?

jamezp commented 6 months ago

So we can just set release to 17 in the compiler plugin and drop the enforcer plugin?

Yes, we could do that as well. The only reason for the enforcer plugin is to know early if you're on the right JDK. It's in no way critical.

jansupol commented 5 months ago

Here it says:

For the Jakarta EE Platform (Platform, Web and Core), the Java compiler --release option is 17. For the component specs, the Jakarta EE Platform requires the Java compiler --release option is at most 17, but component specifications can decide on a lower level.

jamezp commented 5 months ago

Given the proposal of 3.2, I'm going to close this. We can re-address it if plans change.