Skip to content

infra(repo): add CODEOWNERS and branch protection ruleset for main#157

Merged
marc0olo merged 2 commits into
mainfrom
infra/branch-protection
Apr 28, 2026
Merged

infra(repo): add CODEOWNERS and branch protection ruleset for main#157
marc0olo merged 2 commits into
mainfrom
infra/branch-protection

Conversation

@marc0olo
Copy link
Copy Markdown
Member

@marc0olo marc0olo commented Apr 28, 2026

Summary

  • Adds .github/CODEOWNERS assigning @dfinity/dx as the default code owner for all files
  • Documents the accompanying branch ruleset applied directly to repo settings (ruleset ID: 15658343)

What changed

Branch ruleset — "Protect main" (active)

Applied via the GitHub Rulesets API. Rules enforced on main:

Rule Setting
Direct pushes Blocked — every change must go through a PR
Force pushes Blocked
Branch deletion Blocked
Required approvals 1, from a code owner (@dfinity/dx)
Stale review dismissal Enabled — new commits invalidate existing approvals

Bypass actors: @dfinity/dx — team members can merge their own PRs without a separate review.

CODEOWNERS

All files (*) are owned by @dfinity/dx. Every PR will automatically request a review from the team, and the ruleset's "require code owner review" rule will be satisfied by any dx team member approving.

Automated workflows

The sync-motoko workflow opens PRs but never auto-merges, so it is unaffected. Those PRs will require a normal review from any dx member before merging.

Sync recommendation

hand-written

@marc0olo marc0olo merged commit 3e63af4 into main Apr 28, 2026
1 check passed
@marc0olo marc0olo deleted the infra/branch-protection branch April 28, 2026 13:07
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