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

Revert maven plugin upgrade #186

Merged
merged 2 commits into from
May 21, 2023

Conversation

MarkEWaite
Copy link
Contributor

Revert maven plugin upgrade

Testing done

https://docs.github.com/en/code-security/dependabot/dependabot-version-updates/configuration-options-for-the-dependabot.yml-file#ignore
reviewed in hopes of getting the correct syntax to disable updates of
the maven plugin.

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

jenkinsci/bom#1978 notes that the upgrade of
the optional dependency on the maven plugin from 3.16 to 3.21 caused
the Jenkins plugin bill of materials to need an explicit inclusion of
the Jenkins maven plugin into the plugin BOM.

That's not a desired change because the Jenkins maven plugin tests do
not pass when run in the Jenkins plugin BOM test suites.  We don't
want to invest the effort in the Jenkins maven plugin tests to make
them work in the Jenkins plugin BOM.

The jenkinsci#174
dependabot proposed upgrade is being reverted to avoid the problem.
When dependabot next proposes an upgrade of this dependency, it can be
closed or evaluated as an incremental build in the Jenkins plugin BOM
before the upgrade is approved.

This reverts commit 99b537a.
@MarkEWaite MarkEWaite requested a review from a team as a code owner May 21, 2023 04:31
@MarkEWaite MarkEWaite added the bug label May 21, 2023
@MarkEWaite MarkEWaite enabled auto-merge (squash) May 21, 2023 04:32
@MarkEWaite MarkEWaite merged commit 176d98f into jenkinsci:master May 21, 2023
@MarkEWaite MarkEWaite deleted the revert-maven-plugin-upgrade branch August 3, 2023 18:11
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.

1 participant