Skip to content

ci: skip claude-review for the standards-sync bot - #193

Merged
kyle-sexton merged 3 commits into
mainfrom
ci/skip-claude-review-for-standards-sync-bot
Jul 18, 2026
Merged

ci: skip claude-review for the standards-sync bot#193
kyle-sexton merged 3 commits into
mainfrom
ci/skip-claude-review-for-standards-sync-bot

Conversation

@kyle-sexton

Copy link
Copy Markdown
Contributor

Adds melodic-standards-sync[bot] to the claude-review caller's skip-actors (restating the reusable's default member dependabot[bot], since passing the input replaces the default). Today a sync PR's review job spins up and no-ops — the bot is neither allowed nor skipped. A clean skip saves fleet time and Claude tokens on mechanical config syncs, which remain covered by CI, do-not-merge, and pr-issue-linkage gates.

No linked issue.

Related

🤖 Generated with Claude Code

https://claude.ai/code/session_01YZxBP1B8Hf7ZRaqDrP6ma9

Sync PRs from melodic-standards-sync[bot] are neither in allowed-bots nor
skip-actors, so the review job spins up and no-ops instead of skipping
cleanly. Mechanical config syncs carry no review signal worth Claude
tokens; the CI, do-not-merge, and linkage gates still cover them.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YZxBP1B8Hf7ZRaqDrP6ma9

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6e7feaceab

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/claude-review.yml
…osition baseline

The governed-class baseline said claude-review runs on every non-fork
PR; the skip-actors addition in this PR makes sync-bot PRs a second
intentional exclusion. Record it so audits check against the intended
composition, not an obsolete one.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01KCBZ87tV7mgSR6p1nHC9Ft
@kyle-sexton

Copy link
Copy Markdown
Contributor Author

Status: BLOCKED on #200. Required checks passed on the previous run; the fresh run on eaee52d fails ci-status only on the repo-wide visibility mismatch (visibility evidence is public, but .github/runner-policy.json declares private). Codex thread addressed in eaee52d and resolved. After #200 merges, a fresh run (label toggle or push) should go green; then this merges.

…-for-standards-sync-bot

# Conflicts:
#	.github/workflows/claude-review.yml

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 66509ab6ff

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/claude-review.yml
@kyle-sexton kyle-sexton added do-not-merge Hard merge gate: do not merge while applied. and removed do-not-merge Hard merge gate: do not merge while applied. labels Jul 18, 2026
@kyle-sexton
kyle-sexton merged commit 7b8bac8 into main Jul 18, 2026
43 of 44 checks passed
@kyle-sexton
kyle-sexton deleted the ci/skip-claude-review-for-standards-sync-bot branch July 18, 2026 20: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