Skip to content

feat(safe-settings): require code owner review for non-code repos - #196

Merged
marcusburghardt merged 1 commit into
complytime:mainfrom
marcusburghardt:feat/require-codeowner-review
Jul 30, 2026
Merged

feat(safe-settings): require code owner review for non-code repos#196
marcusburghardt merged 1 commit into
complytime:mainfrom
marcusburghardt:feat/require-codeowner-review

Conversation

@marcusburghardt

Copy link
Copy Markdown
Member

Summary

Enable require_code_owner_review on the non-code repos ruleset (community, complytime, complytime-demos, website) to enforce CODEOWNERS approval on all managed repositories. The code repos ruleset already has this enabled.

Also removes complytime-demos from the migration cleanup list since its legacy repo-level ruleset has been deleted.

Changes

  • safe-settings/settings.yml:
    • require_code_owner_review: falsetrue in the "safe-settings: non-code repos" ruleset
    • Removed complytime-demos from the migration cleanup comment block

Notes

  • All managed repos already have CODEOWNERS files in place (9 in .github/CODEOWNERS, org-infra at root).
  • The community repository still has its legacy repo-level ruleset ("Branch Protection") active. It should be kept until this PR is merged and the org-level ruleset is synced with the updated configuration.
  • After merging, run the safe-settings sync workflow targeting the non-code repos to apply the change.

Enable require_code_owner_review on the non-code repos ruleset
(community, complytime, complytime-demos, website) to enforce
CODEOWNERS approval on all managed repositories.

Also remove complytime-demos from the migration cleanup list since
its legacy repo-level ruleset has been deleted.

Assisted-by: OpenCode (claude-opus-4-6)
Signed-off-by: Marcus Burghardt <maburgha@redhat.com>

@gxmiranda gxmiranda left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@marcusburghardt
marcusburghardt merged commit 616d20c into complytime:main Jul 30, 2026
13 checks passed
@marcusburghardt
marcusburghardt deleted the feat/require-codeowner-review branch July 30, 2026 10:51
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