Skip to content

Refresh plugin for December 2022#129

Merged
jtnord merged 1 commit into
jenkinsci:masterfrom
basil:pom
Feb 14, 2023
Merged

Refresh plugin for December 2022#129
jtnord merged 1 commit into
jenkinsci:masterfrom
basil:pom

Conversation

@basil
Copy link
Copy Markdown
Member

@basil basil commented Dec 16, 2022

Combines #98, #100, #105, #109, #114, #116, #120, #121, #123, #127, and #128 into one PR for easy testing and also crosses the Java 11 flag day. Does not include #118 which is a risky upgrade that needs significant testing.

@basil
Copy link
Copy Markdown
Member Author

basil commented Dec 16, 2022

Used my admin powers to replay the CI build with the Jenkinsfile changes: https://ci.jenkins.io/job/Plugins/job/okhttp-api-plugin/job/PR-129/6/ CC @jenkinsci/okhttp-api-plugin-developers

@basil
Copy link
Copy Markdown
Member Author

basil commented Dec 16, 2022

Tested this downstream in github-api-plugin and github-branch-source-plugin. Apart from needing to fix Enforcer warnings related to Kotlin (which I did by importing kotlin-bom downstream), the upgrade passed.

@jtnord jtnord merged commit 9d29e36 into jenkinsci:master Feb 14, 2023
@basil basil deleted the pom branch February 14, 2023 17:04
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.

2 participants