Skip to content

ci(review): configure CodeRabbit reviews - #26

Merged
icavan merged 1 commit into
inclusionAI:mainfrom
icavan:icavan/configure-coderabbit-review
Nov 11, 2026
Merged

icavan merged 1 commit into
inclusionAI:mainfrom
icavan:icavan/configure-coderabbit-review

Conversation

@icavan

@icavan icavan commented Nov 11, 2026

Copy link
Copy Markdown
Collaborator

Outcome

  • Add repository-level CodeRabbit configuration for automatic, incremental review of non-draft PRs.
  • Load ASpec engineering guidelines from AGENTS.md, CONTRIBUTING.md, ARCHITECTURE.md, and DECISIONS.md.
  • Add focused review guidance for planning/evidence, distributed ownership, training semantics, kernels, compatibility, tests, docs, recipes, and CI.
  • Use the repository Ruff configuration alongside ShellCheck and Actionlint; disable duplicate Pylint, Flake8, and Markdownlint feedback.
  • Keep automatic approvals and request-changes behavior disabled.

Verification

  • Checks actually run and results:
    • Ruby YAML safe-load: passed.
    • check-jsonschema against the official CodeRabbit v2 schema: passed.
    • Trailing-whitespace check: passed.
  • GPU/runtime evidence, if relevant: not applicable; configuration-only change.
  • Unverified scope / known limitations: live review execution requires the CodeRabbit GitHub App to have access to this repository.

Compatibility and provenance

  • Public config/checkpoint/export implications: none.
  • Upstream source/revision, source-header/reference and license-text updates, if applicable: none; no upstream implementation content was copied or adapted.

@icavan
icavan requested review from Longxmas and zhouaihui November 11, 2026 14:24

@Longxmas Longxmas left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@icavan
icavan merged commit 9662975 into inclusionAI:main Nov 11, 2026
2 checks passed
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.

2 participants