-
Notifications
You must be signed in to change notification settings - Fork 176
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
[JENKINS-41631] Demonstration of POM update #113
[JENKINS-41631] Demonstration of POM update #113
Conversation
This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation. |
Was building with JDK 7. |
pom.xml
Outdated
@@ -4,7 +4,8 @@ | |||
<parent> | |||
<groupId>org.jenkins-ci.plugins</groupId> | |||
<artifactId>plugin</artifactId> | |||
<version>2.23</version> | |||
<version>2.31-20170612.235547-1</version> <!-- TODO https://github.com/jenkinsci/plugin-pom/pull/67 --> |
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.
If you don't need this right away, do you want to wait until the next version is cut? This plugin isn't released frequently, so it might be worth waiting until the plugin-pom is updated.
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.
Certainly I was not proposing a merge unless and until the dependency is merged and released. That just applies to any use of snapshots.
Really this PR is not intended to accomplish anything in particular except to demonstrate that the upstream PR is consumable.
Target release date - July 10 |
@reviewbybees done |
Downstream of jenkinsci/plugin-pom#67.
@reviewbybees