jira-plugin
jira-plugin copied to clipboard
Add Jira Issue update step to pipeline
Coverage increased (+0.5%) to 48.215% when pulling 69621d8ec55fb440fd5ee2bf70969752e1874641 on aatarasoff:pipeline-jira-issue-updater into a79b0fd5b002cd6aea2055d21934016bd96d1150 on jenkinsci:master.
Looks good to me as a code. But I'm not sure what's the problem with the original JiraIssueUpdateBuilder. It implements SimpleBuildStep, hence it should be available in Pipeline
I don't know how builder will look in pipeline, but I think that notation like:
jiraIssueUpdate(
jqlSearch: jql,
workflowActionName: action,
comment: "Some comment"
)
is really easier for use. Could you provide an example with JiraIssueUpdateBuilder?
And in pipeline syntax
help I didn't see this builder.
Add new pipeline steps. One for updating issue field and one for strict workflow process, cause issue updater set build only Unstable, but not Failure.
Coverage increased (+1.0%) to 48.712% when pulling da4fac3de32bf869d532fa1f1adf47aa3920a9c2 on aatarasoff:pipeline-jira-issue-updater into a79b0fd5b002cd6aea2055d21934016bd96d1150 on jenkinsci:master.
I'll take a look at this tomorrow because it looks like some of the code is duplicated from the original JiraIssueUpdateBuilder
class which as @oleg-nenashev pointed out should be usable already as a Pipeline step.
On the other hand, I would warmly welcome the IssueFieldUpdateStep
functionality as a separate PR!
Please finish work on this so it can be merged, it would enable pipelines for our organization. This is the missing step.
@eskobaba what about https://github.com/jenkinsci/jira-plugin/pull/101#issuecomment-226165515? did you try this one?
I need pipeline compatibility to post a value to a customfield Have i overlooked something?
@aatarasoff
I don't know how builder will look in pipeline, but I think that notation like:
jiraIssueUpdate( jqlSearch: jql, workflowActionName: action, comment: "Some comment" )
is really easier for use. Could you provide an example with JiraIssueUpdateBuilder? And in
pipeline syntax
help I didn't see this builder.
This argument is quite convincing for me, but this PR needs a rebase. Would you have some time for it? if not, I'll try to take it over.
@aatarasoff are you still using this construct? if so, could you rebase it to the current version?