-
Notifications
You must be signed in to change notification settings - Fork 19
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
feat: [CI-14390]: Added new enhancements to plugins/jira
#26
Conversation
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.
also, add unit tests for the changes in plugin_test.go and util_test.go respectively
plugin/plugin.go
Outdated
State: state, | ||
UpdateSequenceNumber: args.Build.Number, | ||
References: []struct { |
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.
add a check to make sure the branch and URI values exist before adding them to the References so it'll only be added when the necessary values are available
plugins/jira
util_test.go updated with unit tests for new functions. In plugin.go not implemented any new
|
Review comments addressed and updated the new patch. |
…rus for printing log messages
@Ompragash @jamie-harness I have addressed the review comments. Please take a look at the latest changes. |
This review covers all Jira issues #21,#22,#23,#24 as per jira spec.
There are some commented lines which are used for debugging. once the review is approved, will remove them.