publish: qa approval detail on the PR, terse verdict on the issue #88

Merged

ffaerber merged 1 commits from fix/qa-verdict-terse into main 2026-07-06 10:10:51 +02:00

Conversation 0 Commits 1 Files Changed 1

+9 -3

ffaerber commented 2026-07-06 10:10:13 +02:00

Owner

Copy Link

Copy Source

Seen on issue #83: @qa's approval pasted technical review detail (whitespace nits) into the issue thread. Review internals belong on the PR; the issue gets exactly ✅ Reviewed PR #N — looks good.

Seen on issue #83: @qa's approval pasted technical review detail (whitespace nits) into the issue thread. Review internals belong on the PR; the issue gets exactly `✅ Reviewed PR #N — looks good.`

ffaerber added 1 commit 2026-07-06 10:10:13 +02:00

publish: qa APPROVE detail goes to the PR — issue gets the terse verdict only ... e22a4cf98c

Seen on issue #83: qa's approval pasted whitespace-nit detail into the issue
thread. The issue is for the creator/orchestration; review internals belong on
the PR. On APPROVE the full reply (+run report) now posts to the PR and the
issue gets exactly '✅ Reviewed PR #N — looks good.' (falls back to the issue
only if no PR could be resolved).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

ffaerber merged commit 36c374d746 into main 2026-07-06 10:10:51 +02:00

ffaerber deleted branch fix/qa-verdict-terse 2026-07-06 10:10:51 +02:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

autopilot

run agents without asking

No labels

Milestone

No items

No Milestone

Projects

Clear projects

No projects

Assignees

Clear assignees

ffaerber hermes intern junior lead ops (Ops Agent) pm qa (QA Agent) senior

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: gitea/agents#88