Skip to content

chore: route reviews to maintainer team - #1441

Merged
alfred-openspec merged 2 commits into
mainfrom
codex/maintainer-approvals
Jul 27, 2026
Merged

chore: route reviews to maintainer team#1441
alfred-openspec merged 2 commits into
mainfrom
codex/maintainer-approvals

Conversation

@TabishB

@TabishB TabishB commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

What changed

  • route repository-wide code ownership to @Fission-AI/openspec-maintainers
  • add Clay Good to the documented core maintainers

Why

Review requests currently route only to Tabish. The new visible maintainer team has Maintain access to OpenSpec and is configured to auto-assign one reviewer using load balancing.

Impact

Once merged, new ready-for-review pull requests will route to an available maintainer instead of always requesting Tabish. The existing one-approval policy remains unchanged.

Validation

  • git diff --check
  • verified the team is visible
  • verified the team has Maintain repository access
  • verified auto-assignment is enabled for one load-balanced reviewer

Summary by CodeRabbit

  • Documentation
    • Updated the project’s core maintainer listing to include Clay Good.
  • Chores
    • Updated repository code ownership defaults to reflect the current maintainers.

@coderabbitai

coderabbitai Bot commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: a6c80abd-762c-48a6-b02c-28a67a2dd437

📥 Commits

Reviewing files that changed from the base of the PR and between 22533a4 and c947037.

📒 Files selected for processing (1)
  • MAINTAINERS.md
🚧 Files skipped from review as they are similar to previous changes (1)
  • MAINTAINERS.md

📝 Walkthrough

Walkthrough

Repository governance metadata is updated by changing the default code owner and adding Clay Good to the Core Maintainers table.

Changes

Repository governance

Layer / File(s) Summary
Ownership and maintainer records
.github/CODEOWNERS, MAINTAINERS.md
The default code owner is changed to the maintainers team, and Clay Good is added as a Core Maintainer.

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

Possibly related PRs

Suggested reviewers: clay-good

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly reflects routing reviews to the maintainer team, which is the main code ownership change.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/maintainer-approvals

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.

@TabishB
TabishB marked this pull request as ready for review July 27, 2026 08:41

@alfred-openspec alfred-openspec 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.

Reviewed the CODEOWNERS and maintainer-list updates, including the automation-maintainer entry. All required checks are green.

@alfred-openspec
alfred-openspec added this pull request to the merge queue Jul 27, 2026
Merged via the queue into main with commit c33fcb3 Jul 27, 2026
16 checks passed
@alfred-openspec
alfred-openspec deleted the codex/maintainer-approvals branch July 27, 2026 11:55
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