Skip to content

docs(rules): run the review process as rules#125

Merged
zolotokrylin merged 14 commits into
docs/rules-pr-requirementsfrom
docs/rules-review
Jul 15, 2026
Merged

docs(rules): run the review process as rules#125
zolotokrylin merged 14 commits into
docs/rules-pr-requirementsfrom
docs/rules-review

Conversation

@zolotokrylin

Copy link
Copy Markdown
Member

Final PR in the CONTRIBUTING-to-rules conversion. Migrates the Review Process into the DEV-4xx category and finishes CONTRIBUTING as a narrative index over the rules.

New rules:

  • DEV-410 reject with Request Changes for objective problems
  • DEV-420 scout open PRs when idle
  • DEV-430 deliver bug-free work; review is a safety check

With this, all four categories exist (21 rules), CONTRIBUTING is down from 441 to ~120 lines of connective narrative, and its intro states the rules are the source of truth when the guide and a rule disagree.

Stacked on #124.

@coderabbitai

coderabbitai Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f03bdaef-8b0d-4e28-8320-e9e850d06e77

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

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

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@holdex

holdex Bot commented Jul 14, 2026

Copy link
Copy Markdown

Time Submission Status

Member # Time Running Total Status Last Update
zolotokrylin 22min ✅ Submitted Jul 15, 2026, 2:15 AM

Submit or update total time with:

@holdex pr submit-time 2h

Add time on top of previous submission with:

@holdex pr add-time 1h30m

See available commands to help comply with our Guidelines.

@zolotokrylin
zolotokrylin merged commit e6aca17 into docs/rules-pr-requirements Jul 15, 2026
2 checks passed
@zolotokrylin
zolotokrylin deleted the docs/rules-review branch July 15, 2026 02:13
@zolotokrylin

Copy link
Copy Markdown
Member Author

@holdex pr add-time 22m

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