From ce7a900737dbf36958ebf7b800eaf68b5c0a32d9 Mon Sep 17 00:00:00 2001 From: claw Date: Wed, 13 May 2026 14:23:50 -0700 Subject: [PATCH] fix(vcs): address self-review NITs - gofmt alignment and comment clarity --- github/review.go | 2 +- vcs/types.go | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/github/review.go b/github/review.go index d6e8243..510a13b 100644 --- a/github/review.go +++ b/github/review.go @@ -102,7 +102,7 @@ func (c *Client) PostReview(ctx context.Context, owner, repo string, number int, // specifies a different non-empty CommitID, we reject the request. for _, comment := range req.Comments { if comment.CommitID != "" { - if payload.CommitID == "" { + if payload.CommitID == "" { // only reachable when req.CommitID is empty payload.CommitID = comment.CommitID } else if payload.CommitID != comment.CommitID { return nil, ErrConflictingCommitIDs diff --git a/vcs/types.go b/vcs/types.go index 5f5a5e1..815fefe 100644 --- a/vcs/types.go +++ b/vcs/types.go @@ -93,7 +93,7 @@ type ReviewRequest struct { // Body is the top-level review comment. Body string `json:"body"` // Event is the review action (approve, request changes, or comment). - Event ReviewEvent `json:"event"` + Event ReviewEvent `json:"event"` // CommitID anchors the review to a specific commit SHA. // Both GitHub and Gitea accept this at the review level. // If empty, each platform applies its default behavior (Gitea uses PR