Skip to content

Only support Java 21 build#1186

Merged
gounthar merged 2 commits into
jenkins-infra:mainfrom
jonesbusy:feature/update-deps
Aug 9, 2025
Merged

Only support Java 21 build#1186
gounthar merged 2 commits into
jenkins-infra:mainfrom
jonesbusy:feature/update-deps

Conversation

@jonesbusy

Copy link
Copy Markdown
Collaborator

Only support Java 21 build

Testing done

mvn clean install and CI

Submitter checklist

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests - that demonstrates feature works or fixes the issue

@jonesbusy jonesbusy added the chore label Aug 9, 2025
@gounthar gounthar enabled auto-merge (squash) August 9, 2025 06:15
@jonesbusy

Copy link
Copy Markdown
Collaborator Author

Weird still some issues

i.j.t.p.core.impl.MavenInvoker # Caused by: java.lang.UnsupportedClassVersionError: io/jenkins/tools/pluginmodernizer/core/recipes/AddDetachedPluginDependency has been compiled by a more recent version of the Java Runtime (class file version 65.0), this version of the Java Runtime only recognizes class file versions up to 61.0 

There is still some Java 17 somewhere

@jonesbusy

Copy link
Copy Markdown
Collaborator Author

Oh I see I miss

// Minimum JDK to run openrewrite
plugin.withJDK(JDK.JAVA_17);

Perhaps it should be a boolean on the JDK class instead of hardcoded on the code

@jonesbusy jonesbusy force-pushed the feature/update-deps branch from ef8ef10 to 62a23eb Compare August 9, 2025 06:47
@jonesbusy

Copy link
Copy Markdown
Collaborator Author

Should be better with 62a23eb

@jonesbusy jonesbusy force-pushed the feature/update-deps branch from 62a23eb to 72d38e2 Compare August 9, 2025 07:05
@gounthar gounthar merged commit 0265cef into jenkins-infra:main Aug 9, 2025
17 checks passed

@gounthar gounthar left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@jonesbusy jonesbusy deleted the feature/update-deps branch August 9, 2025 07:22
@jonesbusy jonesbusy added the enhancement For changelog: Minor enhancement. use `major-rfe` for changes to be highlighted label Aug 9, 2025
gounthar added a commit to gounthar/plugin-modernizer-tool that referenced this pull request Aug 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore enhancement For changelog: Minor enhancement. use `major-rfe` for changes to be highlighted

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants