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

Test on JDK 11 #212

Closed
wants to merge 1 commit into from
Closed

Test on JDK 11 #212

wants to merge 1 commit into from

Conversation

NotMyFault
Copy link
Member

@NotMyFault NotMyFault commented Nov 21, 2022

Let's test it on Java 11 too.

@sghill
Copy link

sghill commented Nov 21, 2022

I had something like this originally, but it missed some important edge cases. Now this plugin uses Gradle's toolchains to test against 8 and 11 here:

for (javaVersion in listOf(8, 11)) {

I think the toolchains approach is more representative because we do compile and release with 8, but users can run with 11.

sghill added a commit that referenced this pull request Nov 23, 2022
@NotMyFault
Copy link
Member Author

I think the toolchains approach is more representative because we do compile and release with 8, but users can run with 11.

I see, I'd like to see Java 11 added to the Jenkinsfile test cases too, given Java 11 is the Jenkins JDK baseline.

@sghill
Copy link

sghill commented Nov 23, 2022

@NotMyFault All integration tests (that run Gradle via GradleTestKit) are being run against Java 11 on Jenkins as of 4810e24. This can be seen here.

@sghill sghill closed this Nov 23, 2022
@sghill
Copy link

sghill commented Nov 23, 2022

I see, I'd like to see Java 11 added to the Jenkinsfile test cases too, given Java 11 is the Jenkins JDK baseline.

We can do this, but it seems to make more sense for Jenkins plugins than this Gradle plugin. To that end, perhaps we shouldn't be calling buildPluginWithGradle at all since we're not producing something to run in Jenkins here. Could you help me better understand the value of compiling against Java 11?

I don't believe Gradle has announced dropping Java 8 for any future release yet. Even if they do, the goal is to retain backwards compatiblity to Gradle 6.3 as long as possible, so it would likely be years after Gradle abandoned 8 that the jpi plugin would abandon 8.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants