Add isRequired parameter to repo_update_pull_request_reviewers tool
#899
ThijsOnGitHub
started this conversation in
Backlog of Suggestions
Replies: 3 comments
|
Currently the logic for adding reviewers is adding or removing, it does update. Add does not have a IsRequired property as part of the add reviewer API. We are going to close this ticket and move it to the discussion backlog for future consideration |
0 replies
|
IMO we should have 3 separate tools. Add, Remove, & Update. We should not have a single tool. We need to look at this and try and make this better in the future. cc: @nikolapeja6 |
0 replies
|
Has there been any updates in this regard? We'd love to make the reviewers added by the agent required. |
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Summary
Enhance the
repo_update_pull_request_reviewerstool to support marking reviewers as required when adding them to a pull request.Tools
Update the following tool with the
isRequiredparameter:repo_update_pull_request_reviewers: Add or remove reviewers for an existing pull request.Endpoint: PUT https://dev.azure.com/{organization}/{project}/_apis/git/repositories/{repositoryId}/pullRequests/{pullRequestId}/reviewers/{reviewerId}?api-version=7.2-preview.1
Current parameters:
repositoryId(required): The ID of the repositorypullRequestId(required): The ID of the pull requestreviewerIds(required): List of reviewer IDs to add or removeaction(required): "add" or "remove"Missing parameter:
isRequired(optional, default: false): When true, marks the reviewer as a required reviewer who must approve before the PR can be completedRules
All reactions