Add isRequired parameter to repo_update_pull_request_reviewers tool
#899
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 |
Beta Was this translation helpful? Give feedback.
-
|
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 |
Beta Was this translation helpful? Give feedback.
-
|
Has there been any updates in this regard? We'd love to make the reviewers added by the agent required. |
Beta Was this translation helpful? Give feedback.
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
Beta Was this translation helpful? Give feedback.
All reactions