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

Use same version in BOM as is declared for plugin #1114

Merged
merged 1 commit into from
Nov 1, 2024

Conversation

MarkEWaite
Copy link
Contributor

@MarkEWaite MarkEWaite commented Nov 1, 2024

Use same version in BOM as is declared for plugin

Previously the plugin required Jenkins 2.452.4 but only used plugin BOM dependencies from 2.440.x.

Also removes the unnecessary declaration of the httpcomponents-client-5 plugin version since the plugin BOM now includes the correct version.

Replaces pull request:

Testing done

Confirmed that mvn dependency:tree reports the same version of httpcomponents-client-5 plugin from the plugin BOM as is currently delivered to users.

Confirmed that automated tests pass.

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

Previously the plugin required Jenkins 2.452.4 but only used plugin BOM
dependencies from 2.440.x.

Also removes the unnecessary declaration of the httpcomponents-client-5
plugin version since the plugin BOM now includes the correct version.

Confirmed that `mvn dependency:tree` reports the same version of
httpcomponents-client-5 plugin from the plugin BOM as is currently
delivered to users.
@MarkEWaite MarkEWaite requested a review from a team as a code owner November 1, 2024 16:21
@MarkEWaite MarkEWaite added the chore Reduce effort required to maintain label Nov 1, 2024
@MarkEWaite
Copy link
Contributor Author

@gounthar I saw that you approved #1113 . Are you OK with using this as a replacement for it?

@gounthar
Copy link
Contributor

gounthar commented Nov 1, 2024

Sure thing, Mark, thanks for asking. 🤗

@MarkEWaite MarkEWaite merged commit 19bde0a into jenkinsci:master Nov 1, 2024
17 checks passed
@MarkEWaite MarkEWaite deleted the use-correct-bom-version branch November 1, 2024 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Reduce effort required to maintain
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants