Repository navigation
Run a plain /review only on an open pull request - #144
Merged
Merged
Conversation
…comes job needs no environment
![ncode-review[bot]](https://avatars.githubusercontent.com/u/12114478?s=60&v=4){kind=small}
There was a problem hiding this comment.
🤖 NCode Review at 927419c: 1 new finding on the changed lines.
|
Superseded by the review at d946379. |
…etup note, and test their closed-pull-request exemption
Contributor
Author
|
/review |
🤖 Review completeVerdict: ✅ Approve — Re-review of what changed since the last review at 927419c: 5 hunks in 3 files and changes taken back in 2 files, by correctness only. No further findings above nits. 1 reviewer run — 0 findings (run details)
Trial: findings are posted as review threads and in the check run. $0.03 on Amazon Bedrock, 7s, at d946379. (how to interact) 📋 OutcomesMerged at d946379: 1 fixed, 0 dismissed, 0 acknowledged, 0 ignored.
|
polewskm
marked this pull request as ready for review
November 1, 2026 09:06
This was referenced Nov 1, 2026
Merged
Merged
polewskm
added a commit
that referenced
this pull request
Nov 1, 2026
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up to #143, from the bot's reviews of the consumer rollout PRs (for example NCodeGroup/Base64Url#21, NCodeGroup/Mediator#21 and NCodeGroup/NCode.Disposables#24):
/reviewnow runs only on an open pull request. A/reviewon a pull request that just closed could cancel theoutcomesrun, or displace it while it waited, because the two share a concurrency group./review explainand/review teachstill answer on a closed pull request, since each has a group of its own. A test asserts the guard in both the workflow and the README's caller.outcomesneeds no environment: the app key is an organization secret, and thecode-reviewenvironment only lets Anthropic's federation rule trust the review job.ifcomment no longer says the command must be exactly/review.Refs #97