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

Modernize to Jenkins 2.375.4 #80

Merged

Conversation

sghill-rewrite
Copy link
Contributor

Hi!

This PR aims to modernize tooling and move this plugin to the recommended Jenkins baseline version.

Testing done

Ran mvn clean verify.

Submitter checklist

Preview Give feedback

Use this link to re-run the recipe: https://app.moderne.io/recipes/org.openrewrite.jenkins.ModernizePlugin

@sghill
Copy link
Contributor

sghill commented Aug 5, 2023

Requires building on Java 11+ to go green (#81)

<dependency>
<groupId>io.jenkins.tools.bom</groupId>
<artifactId>bom-2.375.x</artifactId>
<version>2198.v39c76fc308ca</version>
Copy link
Contributor

Choose a reason for hiding this comment

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

Are you sure this is the correct version to use?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, it's the most recent release of bom-2.375.x, which corresponds to the jenkins.version above. I see you have dependabot enabled, so that will take care of bumping this version on new releases.

The recommended baseline changes over time and older artifacts stop being published. Ideally sometime in the next quarter we'd bump jenkins.version to 2.387.3 and this bom to the latest release of bom-2.387.x.

Happy to send a PR for that down the road if it helps.

@tomasbjerre tomasbjerre merged commit 5073eb3 into jenkinsci:master Aug 5, 2023
@tomasbjerre
Copy link
Contributor

Thanks

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

Successfully merging this pull request may close these issues.

3 participants