-
Notifications
You must be signed in to change notification settings - Fork 98
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
Fix incrementals in PCT #171
Conversation
pom.xml
Outdated
@@ -67,6 +67,7 @@ | |||
<properties> | |||
<revision>2.40</revision> | |||
<changelist>-SNAPSHOT</changelist> | |||
<gitHubRepo>jenkinsci/workflow-durable-task-step-plugin</gitHubRepo> |
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.
or conventionally
<gitHubRepo>jenkinsci/workflow-durable-task-step-plugin</gitHubRepo> | |
<gitHubRepo>jenkinsci/${project.artifactId}-plugin</gitHubRepo> |
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.
true, although I just re-did the incremental tools command, so it should probably do that itself
Co-authored-by: Jesse Glick <[email protected]>
pom.xml
Outdated
@@ -67,6 +67,7 @@ | |||
<properties> | |||
<revision>2.40</revision> | |||
<changelist>-SNAPSHOT</changelist> | |||
<gitHubRepo>jenkinsci/${project.artifactId}-plugin</gitHubRepo> |
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.
This already exists 5 lines down on line 74, added by me in commit 0f45901.
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.
So #165 introduced the property but did not use it; and this should just switch the scm
section. Thanks for catching.
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.
thanks, I missed that, scm.connection was right at the top and when I saw it was wrong I just re-ran incremental tools over it to re-generate the config.
Noticed in https://github.com/jenkinsci/bom/pull/640/checks?check_run_id=3626534874
cc @jglick @basil @car-roll