Skip to content

fix: add CODEOWNERS with catch-all pattern#83

Merged
don-petry merged 2 commits into
mainfrom
claude/issue-62-20260508-1417
May 10, 2026
Merged

fix: add CODEOWNERS with catch-all pattern#83
don-petry merged 2 commits into
mainfrom
claude/issue-62-20260508-1417

Conversation

@don-petry
Copy link
Copy Markdown
Collaborator

@don-petry don-petry commented May 8, 2026

Summary

  • Adds .github/CODEOWNERS with * @petry-projects/org-leads as the default catch-all owner
  • Ensures all files have an owner so require_code_owner_review applies repo-wide
  • Follows the codeowners-standard.md: @petry-projects/org-leads is the first (and only) owner on the catch-all line, using a team slug (no individual users)

Test plan

  • Verify compliance audit no longer raises codeowners-no-catchall for this repo
  • Confirm require_code_owner_review is satisfied by members of @petry-projects/org-leads

Closes #62

Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated repository configuration for code ownership management.

Review Change Stack

Adds .github/CODEOWNERS with a default `* @petry-projects/org-leads`
catch-all so all files have an owner and require_code_owner_review
applies repo-wide.

Closes #62

Co-authored-by: Don Petry <don-petry@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 8, 2026 14:20
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 8, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 983eba06-2736-4370-9cd5-174c6de2cae3

📥 Commits

Reviewing files that changed from the base of the PR and between 6a43ede and 0042efa.

📒 Files selected for processing (1)
  • .github/CODEOWNERS

📝 Walkthrough

Walkthrough

A default catch-all rule is added to .github/CODEOWNERS, assigning ownership of all repository files to @petry-projects/org-leads. This addresses a compliance requirement for establishing universal code owner coverage.

Changes

CODEOWNERS Compliance

Layer / File(s) Summary
Compliance Configuration
.github/CODEOWNERS
Default catch-all rule (*) assigns all repository files to @petry-projects/org-leads for code owner review coverage.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related issues

🚥 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 and specifically describes the main change: adding a CODEOWNERS file with a catch-all pattern to address the compliance issue.
Linked Issues check ✅ Passed The pull request fully implements the requirement from issue #62 by adding a .github/CODEOWNERS file with the catch-all pattern * @petry-projects/org-leads``, resolving the codeowners-no-catchall compliance finding.
Out of Scope Changes check ✅ Passed The pull request contains only the necessary CODEOWNERS file addition with no extraneous or unrelated changes beyond the stated scope of issue #62.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/issue-62-20260508-1417

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 and usage tips.

@don-petry
Copy link
Copy Markdown
Collaborator Author

@don-petry This PR is ready for review. It adds .github/CODEOWNERS with * @petry-projects/org-leads to fix the codeowners-no-catchall compliance finding.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a repository-wide CODEOWNERS catch-all so every file has an owner and branch protections using require_code_owner_review can apply consistently across the repo.

Changes:

  • Added .github/CODEOWNERS with a * catch-all mapping to @petry-projects/org-leads.
  • Included a brief header comment describing the default ownership rule.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@don-petry don-petry disabled auto-merge May 10, 2026 18: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.

Compliance: codeowners-no-catchall

2 participants