Skip to content

ci: update SAM PR Reviewer to latest version#8928

Open
roger-zhangg wants to merge 1 commit intoaws:developfrom
roger-zhangg:update-sam-pr-reviewer-sha-2
Open

ci: update SAM PR Reviewer to latest version#8928
roger-zhangg wants to merge 1 commit intoaws:developfrom
roger-zhangg:update-sam-pr-reviewer-sha-2

Conversation

@roger-zhangg
Copy link
Copy Markdown
Member

@roger-zhangg roger-zhangg commented Apr 17, 2026

Updates pinned SHA for roger-zhangg/sam-pr-reviewer to d015ddfe67119a6a207a7d8260c1535940d9d2ed.

Changes:

  • Skip posting review when no issues found
  • Fix false positives: clarify that workspace is BEFORE state, diff is source of truth

@roger-zhangg roger-zhangg requested a review from a team as a code owner April 17, 2026 23:06
Copy link
Copy Markdown
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review Results

Reviewed: fdf769d..8a944d5
Files: 1
Comments: 0

✅ No issues found. The changes look good.

The PR updates the pinned commit SHA for the roger-zhangg/sam-pr-reviewer GitHub Action. The action continues to use a full 40-character commit SHA (best practice for supply-chain security), permissions and secrets are unchanged, and the # v1 version annotation is preserved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant