Skip to content

Add CODEOWNERS so PRs get a reviewer - #102

Merged
carole-lavillonniere merged 1 commit into
mainfrom
add-codeowners
Sep 2, 2026
Merged

Add CODEOWNERS so PRs get a reviewer#102
carole-lavillonniere merged 1 commit into
mainfrom
add-codeowners

Conversation

@carole-lavillonniere

@carole-lavillonniere carole-lavillonniere commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Especially thinking of dependabot PRs.

The repo has no CODEOWNERS in any of the three locations GitHub reads, and
nothing in dependabot.yml fills in for it (`reviewers` is gone from Dependabot's
options reference; we set no `assignees`). So Dependabot PRs open with no
reviewer and no assignee -- #100 has been sitting that way -- and every other PR
depends on someone noticing it.

@localstack/saas matches where secops already routes this image's CVE findings
(localstack_products.json, team: SaaS), and the one collaborator with push access
who has been reviewing here is in that team.

Same single-glob pattern as secops' CODEOWNERS, which is what auto-requests
core-systems on its Dependabot PRs (e.g. localstack/secops#110), so this is
proven to work for bot PRs in this org.

Note it only auto-requests: main has no branch protection or rulesets, so
nothing blocks a merge without review. Enforcing that needs a ruleset with
"Require review from Code Owners".
@carole-lavillonniere
carole-lavillonniere merged commit 9b87223 into main Sep 2, 2026
1 check 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