Skip to content

Commit b3b578f

Browse files
martinsifraclaude
andcommitted
feat: Add reviewers field to PUT /merge-request/{id}
- New MergeRequestReviewerRepository with diff-based setReviewers() and eligibility validation (delegates to AdminProjectRepository::getReviewers) - New InvalidReviewerException with string code storage.mergeRequests.invalidReviewer - MergeRequestUpdateRequest: optional reviewers field (PATCH semantics — omit to keep) - MergeRequestUpdateProcessor: calls setReviewers when field is present - MergeRequestUpdateAction: catches InvalidReviewerException → HTTP 422 - MergeRequestResponseProvider: includes reviewers array in all MR responses - MergeRequestResponse: OA schema updated - E2E tests: happy-path (add/omit/clear) + invalid reviewer ID Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
1 parent 1e84168 commit b3b578f

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

src/Keboola/StorageApi/Client.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1019,7 +1019,8 @@ public function getMergeRequest(int $mergeRequestId): array
10191019
* description?: ?string,
10201020
* externalId?: ?string,
10211021
* autoMergeStrategy?: ?string,
1022-
* autoMergeAt?: ?string
1022+
* autoMergeAt?: ?string,
1023+
* reviewers?: list<int>
10231024
* } $data
10241025
* @return array mrData
10251026
*/

0 commit comments

Comments
 (0)