Skip to content

Updating pom with latest forensic api plugin version#938

Merged
uhafner merged 1 commit intojenkinsci:mainfrom
ykhandelwal913:main
Mar 15, 2025
Merged

Updating pom with latest forensic api plugin version#938
uhafner merged 1 commit intojenkinsci:mainfrom
ykhandelwal913:main

Conversation

@ykhandelwal913
Copy link
Contributor

Testing done

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

@uhafner uhafner added the dependencies Update of dependencies label Mar 15, 2025
Copy link
Member

@uhafner uhafner left a comment

Choose a reason for hiding this comment

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

This is not necessarily required but anyway... (I will remove the version soon, see TODO)

@uhafner uhafner merged commit 5a4c833 into jenkinsci:main Mar 15, 2025
30 of 31 checks passed
@ykhandelwal913
Copy link
Contributor Author

Thanks for the review. @uhafner .

can I get a new version released so that we can update the coverage plugin Pom.xml as well?

@uhafner
Copy link
Member

uhafner commented Mar 15, 2025

This is not required, Jenkins finds the correct version automatically...

@ykhandelwal913
Copy link
Contributor Author

Reson i asked because we dont install git forensic plugin separately as it is bundled with coveerage plugin, which is pointing to old version.
https://github.com/jenkinsci/coverage-plugin/blob/main/plugin/pom.xml#L30

@uhafner
Copy link
Member

uhafner commented Mar 15, 2025

In Jenkins plugins are never bundled, they are distributed separately.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Update of dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants