Skip to content

ci: establish a shared review baseline and adopt it in Claude reviews #3149

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
ChenhanYu merged 3 commits into from
Oct 30, 2026
+304 −45
Merged

ci: establish a shared review baseline and adopt it in Claude reviews #3149

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
91338d6
docs: establish a shared code review baseline
ChenhanYu Oct 30, 2026
ed090cd
ci: use the shared baseline for Claude reviews
ChenhanYu Oct 30, 2026
ddf3c0a
docs: clarify delegated LGTM approval
ChenhanYu Oct 30, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions .agents/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,13 @@ a copy:
- New skills go in `plugins/modelopt/skills/<skill-name>/SKILL.md`.
- Shared support files go in `plugins/modelopt/skills/common/`.

## Review baseline

[Code review baseline](review-guidelines/code-review.md) is the shared rubric
for future reviews. Read the version at the PR's base revision and report that
commit. Domain-specific rubrics, such as the
[technical blog guideline](review-guidelines/technical-blog.md), add checks to it.

## Project-level cluster config

The remote-execution skills look for a `clusters.yaml` at, in order:
Expand Down
Loading
Loading