-
Notifications
You must be signed in to change notification settings - Fork 166
Bump plugin parent POM #72
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
Conversation
That failure doesn't seem related to this change, since it also occurred in #71. I suspect the same would occur on As far as I can tell, this change will be ready to merge once |
I recommend reviewing this change with the "Hide whitespace changes" option enabled. |
pom.xml
Outdated
<relativePath /> | ||
</parent> | ||
<artifactId>pipeline-utility-steps</artifactId> | ||
<packaging>hpi</packaging> | ||
<version>${revision}${changelist}</version> | ||
<name>Pipeline Utility Steps</name> | ||
<url>https://wiki.jenkins.io/display/JENKINS/Pipeline+Utility+Steps+Plugin</url> | ||
<url>https://github.com/jenkinsci/${project.artifactId}-plugin</url> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To do this change you must first move the docs from the wiki to the readme. But I also don't like putting user docs in the root readme, that file is more imho for visitors of the repo itself. So the docs should be put in for example docs/README.md
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To do this change you must first move the docs from the wiki to the readme.
I have no interest in doing this, so I will revert the change.
This reverts commit d1d3ed4.
Bumps the plugin parent POM from 3.48 to 3.54.
See the changelog and the full diff in the compare view.