Skip to content

Name the explain and teach commands in the code review caller's header - #22

Merged
polewskm merged 1 commit into
mainfrom
code-review-v1
Nov 1, 2026
Merged

polewskm merged 1 commit into
mainfrom
code-review-v1

Conversation

@polewskm

@polewskm polewskm commented Nov 1, 2026

Copy link
Copy Markdown
Contributor

Follow-up to #21, which merged one commit before this one. The caller's header now names /review explain and /review teach, and says both answer on a closed pull request. Its setup note now matches tools/Enable-CodeReview.ps1: the code-review environment holds no secret, and the repository gets access to the organization's two secrets. The if comment says the explain and teach checks must match the concurrency group's. This matches the canonical caller in NCodeGroup/code-review#144.

Refs NCodeGroup/code-review#97

@polewskm
polewskm deployed to code-review November 1, 2026 09:07 — with GitHub Actions Active
@ncode-review

ncode-review Bot commented Nov 1, 2026 •

Copy link
Copy Markdown

🤖 Review complete

Verdict: ✅ Approve — No findings.

3 reviewers run — 0 findings (run details)

Reviewer Model Findings Top
conventions sonnet-5-5 0 🟢
hygiene haiku-5-5 0 🟢
rule-check sonnet-5-5 0 🟢

Skipped: api-compat, architecture, completeness, correctness, data-model, infrastructure, intent, performance, security, tests

Trial: findings are posted as review threads and in the check run. $0.15 on Amazon Bedrock, 17s, at 9866f73. (how to interact)

📋 Outcomes

Merged at 9866f73. No findings were posted as threads.

@polewskm
polewskm marked this pull request as ready for review November 1, 2026 09:12
@polewskm
polewskm deployed to code-review November 1, 2026 09:13 — with GitHub Actions Active
@polewskm
polewskm merged commit ee4fa55 into main Nov 1, 2026
13 checks passed
@polewskm
polewskm deleted the code-review-v1 branch November 1, 2026 09:13

This branch was successfully deployed

1 active deployment
code-review — 9866f738 Deployed Nov 1, 2026 by polewskm via review #46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant