-
Notifications
You must be signed in to change notification settings - Fork 42
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
added release notes file option for create-release v6 #309
Conversation
This pull request has been linked to Shortcut Story #47628: ADO v6 Create Release task doesn't accept content from files. |
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.
Code looks good, just a comment about what we do when both release notes and release notes file is specified for consideration.
@@ -64,6 +66,13 @@ export function createCommandFromInputs(logger: Logger, task: TaskWrapper): Crea | |||
GitCommit: task.getInput("GitCommit"), | |||
}; | |||
|
|||
if (!command.ReleaseNotes) { |
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.
In the cli we fail the command if both release notes and release notes file are specified, I wonder if we should do the same here to prevent customers hitting any unexpected behavior?
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.
Good point, can change this to have the same behaviour
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.
Test for specifying both fields needs updating, but other than that this is 👍 now
fixes #310
[sc-47628]
Specifying both release notes and release notes file throws and error