Skip to content

ci(robin): keep the comment-only review mode (request-changes: false) - #73

Merged
devopam merged 1 commit into
mainfrom
ci/robin-comment-mode
Oct 18, 2026
Merged

devopam merged 1 commit into
mainfrom
ci/robin-comment-mode

Conversation

@devopam

@devopam devopam commented Oct 18, 2026 •

Copy link
Copy Markdown
Owner

Why

This follows up #72. CodeRabbit found it on the same change in devopam/MCPg#357.

Change

request-changes: false in .github/workflows/robin.yml. This keeps the old comment-only mode.

🤖 Generated with Claude Code

https://claude.ai/code/session_01XrLEbkf9BAxrqFTBfcU48z

Summary by Sourcery

Bug Fixes:

  • Preserve comment-only Robin reviews by explicitly disabling change requests, preventing high-severity findings from blocking merges.

Summary by CodeRabbit

  • Chores
    • Review automation remains in comment-only mode and will not automatically request changes. This keeps automated review feedback from changing a pull request’s review status. No changes to the application experience are included in this update.

RetriggerView in GreptileConfidence Score: 5/5

The change appears safe to merge.

Summary

Keeps Robin reviews comment-only by explicitly passing request-changes: false.

  • Robin reviews stay in comment-only mode.

Reviews (1) · Last reviewed commit: "ci(robin): keep the comment-only review ..." · Reviewed by Greptile

Follow-up to #72, from the review of the same change in devopam/MCPg#357:
the reusable workflow sent request-changes false when its boolean input
was unset; the action's own default defers to the repo config, which
defaults to true, so a high finding would post REQUEST_CHANGES and could
block a merge.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01XrLEbkf9BAxrqFTBfcU48z
@sourcery-ai

sourcery-ai Bot commented Oct 18, 2026

Copy link
Copy Markdown

Sorry @devopam, you've used your own review budget of 250,000 diff characters for the last 7 days.

You can request another review in 1 hour and 36 minutes by commenting @sourcery-ai review. Upgrade to get a review now.

@github-actions

github-actions Bot commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

The Robin Hood of code review

✅ Review done. Nothing worth flagging — looks clean to me.

Push fixes whenever you like, then comment /robin for another pass.

Robin v2.12.0 · antongulin/robin@fc0bf91 (fc0bf91) · diff-only review (agent mode off)

@coderabbitai

coderabbitai Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 75f55071-e196-43cc-9839-1d36456b096b
📥 Commits

Reviewing files that changed from the base of the PR and between f986467 and 91b1784.

📒 Files selected for processing: 1
  • .github/workflows/robin.yml

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 3 remain after this review.

📝 Walkthrough

The Robin workflow now explicitly sets request-changes to false and includes comments about the reusable workflow and action defaults.

Changes: Robin review workflow

Layer / File(s) Summary
Set request-changes input
.github/workflows/robin.yml
Comments describe the reusable workflow's omitted-input behavior and the action's default. The workflow sets request-changes to false.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to 91b17

High-severity findings remain comments rather than blocking pull requests. No actionable merge risk remains beyond normal checks.

### Architecture Summary

Architecture risk: 🔵 Low · up to 91b17

The changed surface does not map to a changed system, dependency edge, entrypoint, or external dependency.

Changed systems: None identified.

Architecture concerns
No architecture-level concerns identified.

Review details

Before / after behavior

  • observed — Modified behavior in .github/workflows/robin.yml: The workflow adds comments about the reusable workflow’s omitted-input behavior and the action’s default, then explicitly sets request-changes to false.
Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely identifies the main change: keeping Robin in comment-only review mode by setting request-changes: false.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown
Contributor

Failed to generate code suggestions for PR

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The Robin Hood of code review

Heads up: this is a point-in-time review. Push fixes freely, then comment /robin whenever you want another pass.

✅ No issues found

Summary

The diff adds a request-changes: false setting and three explanatory comments to a GitHub Actions workflow file, explicitly opting out of automatic REQUEST_CHANGES to maintain comment-only review mode. The change is benign, the comments clarify the reasoning for future maintainers, and no new secrets, logic, or runtime behavior is introduced.

@devopam
devopam merged commit 2b3fb72 into main Oct 18, 2026
9 of 10 checks passed
@devopam
devopam deleted the ci/robin-comment-mode branch October 18, 2026 14:59
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