Skip to content

chore: add catch-all CODEOWNERS entry for full path coverage - #346

Merged
robnester-rh merged 1 commit into
conforma:mainfrom
robnester-rh:EC-2150-codeowners
Sep 3, 2026
Merged

robnester-rh merged 1 commit into
conforma:mainfrom
robnester-rh:EC-2150-codeowners

Conversation

@robnester-rh

Copy link
Copy Markdown
Contributor

What

Prepend a * @conforma/devs catch-all to the existing .github/CODEOWNERS.

Why

The file previously assigned owners only to the Conventional Commits enforcement paths, leaving the rest of the repo (and most of .github/) without a code owner. The catch-all gives every path an owner so code-owner review can be required org-wide, while last-match-wins preserves the specific commit-lint gate ownership below it.

Co-Authored-By: Claude noreply@anthropic.com
Ref: EC-2150

Prepend a `* @conforma/devs` catch-all so every path (including all of
.github/) has a code owner, while preserving the existing granular
ownership of the Conventional Commits enforcement files (last-match-wins
keeps those specific rules intact).

Co-Authored-By: Claude <noreply@anthropic.com>
Ref: EC-2150
@robnester-rh
robnester-rh requested a review from a team as a code owner September 2, 2026 14:02
@coderabbitai

coderabbitai Bot commented Sep 2, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Enterprise

Run ID: 75da6a18-8a05-496c-9d83-d53a18ea1895

📥 Commits

Reviewing files that changed from the base of the PR and between 6708bc2 and d4e8cca.

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

Included review availability: Your plan provides up to 12 included reviews per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The CODEOWNERS file now assigns repository-wide ownership to @conforma/devs, including .github/.

Changes

Repository ownership

Layer / File(s) Summary
Default CODEOWNERS ownership
.github/CODEOWNERS
Added a repository-wide rule that assigns @conforma/devs ownership for all paths.

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

Merge Risk: ⚪ Minimal · up to d4e8c

This adds repository-wide code-owner coverage while preserving the existing specific ownership rule; no actionable merge-blocking risk remains beyond normal checks and review.

Suggested reviewers: acepresso

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the addition of a catch-all CODEOWNERS entry for full path coverage.
Description check ✅ Passed The description accurately explains the catch-all entry, its purpose, and the effect of last-match-wins behavior.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

@qodo-for-conforma

Copy link
Copy Markdown

PR Summary by Qodo

Add catch-all CODEOWNERS coverage

⚙️ Configuration changes 🕐 Less than 5 minutes

Grey Divider

AI Description

• Assigns @conforma/devs as the default owner for every repository path.
• Preserves later path-specific rules through CODEOWNERS last-match-wins behavior.
High-Level Assessment

The catch-all rule is the simplest reliable way to guarantee complete ownership coverage. Placing it before specific entries correctly preserves CODEOWNERS precedence; enumerating directories individually would be more error-prone and require ongoing maintenance.

Files changed (1) +3 / -0

Other (1) +3 / -0
CODEOWNERSAdd default ownership for all repository paths +3/-0

Add default ownership for all repository paths

• Adds a catch-all rule assigning every path, including '.github/', to '@conforma/devs'. Existing specific ownership entries remain later in the file and therefore retain precedence.

.github/CODEOWNERS

@qodo-for-conforma

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Tip of the day
💡 Did you know, you can turn on the rule miner and Qodo learns your standards from review history

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

@codecov

codecov Bot commented Sep 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Flag Coverage Δ
unit-tests 55.75% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@robnester-rh
robnester-rh merged commit 89fbae2 into conforma:main Sep 3, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants