Skip to content

AST-171789 Point kics CODEOWNERS at cx-maintainers-kics - #8109

Closed
cx-lior-poterman wants to merge 1 commit into
masterfrom
ast-171789-kics-codeowners-maintainers
Closed

cx-lior-poterman wants to merge 1 commit into
masterfrom
ast-171789-kics-codeowners-maintainers

Conversation

@cx-lior-poterman

Copy link
Copy Markdown
Contributor

Points default code ownership at the cx-maintainers-kics team, aligning CODEOWNERS with the reviewer team required by the kics merge-protection ruleset.

Previously ownership pointed at a different team, which meant a PR needed approvals from two separate teams to satisfy both the ruleset and the code-owner rule. cx-maintainers-kics has push access to the repo, so the ownership rule resolves correctly.

The CODEOWNERS entry pointed at @Checkmarx/kics, a different team than
the reviewer team required by the kics merge-protection ruleset. That
mismatch meant PRs needed approvals from two separate teams to satisfy
both the ruleset and code-owner review. Update the default owner to
@Checkmarx/cx-maintainers-kics, which has push access to the repo and
matches the ruleset's required reviewer team.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@cx-lior-poterman

Copy link
Copy Markdown
Contributor Author

Superseded by #8108, which now includes this CODEOWNERS change alongside the release environment gating.

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