thuanle/claw-email
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Require PR discussion comments in reviewer workflow #17

Merged
thuanle merged 1 commits from fix/reviewer-pr-discussion-comments into main 2026-04-28 06:13:28 +07:00
Conversation 0 Commits 1 Files Changed 2
+4 -1
codex commented 2026-04-28 06:11:54 +07:00
Collaborator
Copy Link
Copy Source

Summary

  • require reviewers to read PR discussion comments before posting a verdict
  • clarify that reviews, inline review comments, and PR discussion comments are separate context sources
  • update reviewer process wording in AGENT.md and .codex/skills/review-pr/SKILL.md

Changes

  • AGENT.md: reviewer step now explicitly includes PR discussion comments
  • .codex/skills/review-pr/SKILL.md: load PR discussion comments during review and block verdict posting until they are checked

Test Plan

  • gofmt -l . — clean
  • go vet ./... — OK
  • go test ./... — OK

Fixes #16

## Summary - require reviewers to read PR discussion comments before posting a verdict - clarify that reviews, inline review comments, and PR discussion comments are separate context sources - update reviewer process wording in `AGENT.md` and `.codex/skills/review-pr/SKILL.md` ## Changes - `AGENT.md`: reviewer step now explicitly includes PR discussion comments - `.codex/skills/review-pr/SKILL.md`: load PR discussion comments during review and block verdict posting until they are checked ## Test Plan - [x] `gofmt -l .` — clean - [x] `go vet ./...` — OK - [x] `go test ./...` — OK Fixes #16
codex added 1 commit 2026-04-28 06:11:54 +07:00
docs: require PR discussion comments in reviews
CI / test (pull_request) Successful in 10m45s
Details
CI / fmt (pull_request) Successful in 4m46s
Details
28224ccb9f
thuanle merged commit 3985b9dae3 into main 2026-04-28 06:13:28 +07:00
thuanle deleted branch fix/reviewer-pr-discussion-comments 2026-04-28 06:13:28 +07:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
claudecode codex opencode thuanle
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: thuanle/claw-email#17