Skip to content

ci: bump claude-review pin to df54d0e - #255

Merged
kyle-sexton merged 1 commit into
mainfrom
ci/claude-review-pin-df54d0e
Jul 17, 2026
Merged

ci: bump claude-review pin to df54d0e#255
kyle-sexton merged 1 commit into
mainfrom
ci/claude-review-pin-df54d0e

Conversation

@kyle-sexton

Copy link
Copy Markdown
Contributor

No related issue: step 3 of melodic-software/standards#159 (six-consumer claude-review pin bump); that issue closes only when all six consumers land.

Related

What

Bump the claude-review.yml reusable-workflow pin 4dbb0dfcdf54d0e8c899baa7e57344c3633f033129da6632.

  • Restores failure-comment visibility (a failed review now surfaces instead of a silent green).
  • The upstream diff is otherwise additive-only: an optional standards-ref input plus two optional App secrets for the private depth tier — unused by this caller, no shape change.
  • The df54d0e runner-input contract is already present in this repo's synced .github/standards/runner-policy/policy.json (or the local component, for standards itself), so the Runner policy lane validates it as-is.

🤖 Generated with Claude Code

https://claude.ai/code/session_01K3QehVwmWzkBLpKokNCkkt

Step 3 of standards#159: restores failure-comment visibility
(fix(claude-review): surface real review failures instead of a silent
green). Upstream diff is otherwise additive-only (optional standards-ref
input + two optional App secrets, unused here). The df54d0e runner-input
contract is already in the synced runner-policy component.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

https://claude.ai/code/session_01K3QehVwmWzkBLpKokNCkkt

Co-authored-by: Claude Fable 5 (1M context) <noreply@anthropic.com>
@kyle-sexton
kyle-sexton merged commit da1f98a into main Jul 17, 2026
19 checks passed
@kyle-sexton
kyle-sexton deleted the ci/claude-review-pin-df54d0e branch July 17, 2026 15:23
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