Skip to content

chore: add CODEOWNERS for CI/CD trusted computing base - #101

Merged
skuenzli merged 1 commit into
mainfrom
chore/add-codeowners
Jun 27, 2026
Merged

chore: add CODEOWNERS for CI/CD trusted computing base#101
skuenzli merged 1 commit into
mainfrom
chore/add-codeowners

Conversation

@skuenzli

Copy link
Copy Markdown
Contributor

Follow-up to the #62 supply-chain hardening (out-of-band item #1).

Adds .github/CODEOWNERS to auto-request maintainer review on PRs touching the trusted CI/CD path:

Path Why
/.github/workflows/ the SHA-pinned, zizmor-linted workflows that build, test, and OIDC-publish to PyPI
/.github/dependabot.yml dependency/automation policy
/.github/CODEOWNERS changing ownership itself requires owner review

Owner choice

@skuenzli — no team (build, product-eng) has direct access to this repo, so a team owner would be flagged invalid by CODEOWNERS validation.

Important: this file is advisory by itself

CODEOWNERS only auto-requests review. To make code-owner review a merge requirement, enable "Require review from Code Owners" in the branch protection rule / ruleset for main (a repo setting, not expressible in the file).

Caveat for a solo maintainer: that toggle would block your own PRs (you can't approve your own), so it's left off for now — the auto-request is still valuable for outside-contributor PRs (e.g. recent template/dep PRs). Revisit if a second reviewer joins.

CI: this is a docs/config-only change; the build matrix + zizmor lint run as usual.

🤖 Generated with Claude Code

Auto-request maintainer review on changes to the workflow files and
dependency/automation policy — the trusted path that builds, tests, and
publishes the package. Follow-up to the GH #62 supply-chain hardening
(out-of-band item #1).

Owner is @skuenzli (no team has direct repo access, so a team owner would
be flagged invalid). The file only auto-requests review; making it a merge
requirement needs the branch-protection "Require review from Code Owners"
toggle, left off for now since a solo maintainer can't review their own PRs.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@skuenzli
skuenzli merged commit 5de5da4 into main Jun 27, 2026
14 of 16 checks passed
@skuenzli
skuenzli deleted the chore/add-codeowners branch June 27, 2026 21:23
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.

1 participant