Skip to content

add CODEOWNERS for default review ownership - #66

Merged
atkrad merged 1 commit into
mainfrom
harden-branch-protection
Aug 3, 2026
Merged

add CODEOWNERS for default review ownership#66
atkrad merged 1 commit into
mainfrom
harden-branch-protection

Conversation

@atkrad

@atkrad atkrad commented Aug 3, 2026

Copy link
Copy Markdown
Member

Summary

  • Add .github/CODEOWNERS with @atkrad as default owner for the whole tree.
  • Prepares Scorecard Branch-Protection for "Require review from Code Owners" later (do not turn that toggle on yet while solo).

Ruleset follow-up (Settings → Rules → Protect main)

  1. Require status checks to pass → add Unit Tests, Integration Tests, E2E Tests
  2. Keep Require branches to be up to date before merging
  3. Optional later: last-push approval / code owners / drop bypass (needs a second human)

Test plan

  • CI green on the PR
  • Finish ruleset save if not already saved
  • Merge, then Scorecard re-run for Branch-Protection alert Review the code #1

Labels

  • One of: kind/feature, kind/bug, kind/docs, kind/chore
  • Add breaking-change if this breaks existing CLI or config behavior
  • Add skip-changelog for internal-only PRs that should not appear in notes

@github-actions github-actions Bot added the area/ci GitHub Actions and release packaging label Aug 3, 2026
@codecov

codecov Bot commented Aug 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@atkrad atkrad added kind/chore Cleanup, deps, CI, or refactor with no user-facing change skip-changelog Exclude this PR from release notes labels Aug 3, 2026
@atkrad
atkrad merged commit 021ad58 into main Aug 3, 2026
17 checks passed
@atkrad
atkrad deleted the harden-branch-protection branch August 3, 2026 22:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ci GitHub Actions and release packaging kind/chore Cleanup, deps, CI, or refactor with no user-facing change skip-changelog Exclude this PR from release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant