Skip to content

docs: add PR workflow rules to CLAUDE.md#102

Merged
let-sunny merged 1 commit intomainfrom
docs/pr-workflow-rules
Mar 26, 2026
Merged

docs: add PR workflow rules to CLAUDE.md#102
let-sunny merged 1 commit intomainfrom
docs/pr-workflow-rules

Conversation

@let-sunny
Copy link
Copy Markdown
Owner

@let-sunny let-sunny commented Mar 26, 2026

Summary

Changes

New ### PR Workflow subsection with 6 rules:

  1. Draft-first PR creation
  2. Draft conversion on changes
  3. subscribe_pr_activity for real-time monitoring
  4. CodeRabbit rate limit handling (empty commit re-trigger)
  5. Immediate review comment addressing
  6. Squash merge with explicit user approval

Made with Cursor

Summary by CodeRabbit

  • Documentation
    • Updated internal PR workflow documentation to establish explicit control-flow procedures. New guidelines cover draft PR practices, real-time review and CI monitoring, handling rate-limit messaging with re-trigger options, systematic review comment processing, and explicit approval enforcement before merging.

Add conventions for draft PR lifecycle, CodeRabbit review monitoring,
rate limit handling, and merge approval process.
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 26, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 9cad4439-037d-46b5-8347-e67656ca7a30

📥 Commits

Reviewing files that changed from the base of the PR and between 6c02581 and 64053e9.

📒 Files selected for processing (1)
  • CLAUDE.md

📝 Walkthrough

Walkthrough

Added a new PR Workflow section to CLAUDE.md documenting an explicit control flow for pull request management, including draft creation, activity monitoring with subscribe_pr_activity, CI/rate-limit handling, review comment addressing, and explicit user approval before merging with squash and branch deletion.

Changes

Cohort / File(s) Summary
Documentation
CLAUDE.md
Added new PR Workflow section describing pull request handling procedures, including draft management, real-time monitoring, rate-limit messaging, and merge approval process.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A workflow so clear, now written in stone,
Draft, review, and merge—step by step, fully known!
CodeRabbit watches, CI confirms the way,
Explicit approval leads to merge day! 📋✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/pr-workflow-rules

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

@let-sunny let-sunny marked this pull request as ready for review March 26, 2026 08:48
@let-sunny let-sunny merged commit f2cbddc into main Mar 26, 2026
2 of 3 checks passed
@let-sunny let-sunny deleted the docs/pr-workflow-rules branch March 26, 2026 08:48
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