Update CODEOWNERS#31342
Conversation
Review⏸️ Automatic reviews for this PR are paused. This PR has already received 2 automatic reviews. To run another review, a codeowner can comment
✅ No style-guide issues found in commit CommandsOnly codeowners can run commands. Post a comment with the command to trigger it.
|
|
This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:
|
There was a problem hiding this comment.
Minor repo maintenance change. One issue to flag:
LOW — PR description is empty (only the template is present). Please add a brief summary; for example: "Add CODEOWNERS entry for the security-center changelog directory." This helps reviewers understand intent without reading the diff.
Change summary: Adds to , mirroring the existing owners for and plus . The directory already exists, so the ownership assignment is correct.
|
Minor repo maintenance change. One issue to flag: LOW — PR description is empty (only the template is present). Please add a brief summary; for example: "Add CODEOWNERS entry for the security-center changelog directory." This helps reviewers understand intent without reading the diff. Change summary: Adds |
|
I've completed the review of PR #31342. Review summary: Issue flagged:
Labels:
The review is posted as a comment on PR #31342. |
|
CI run failed: build logs |
|
Preview URL: https://1ee51f33.preview.developers.cloudflare.com |
99f4f26 to
1ee51f3
Compare
Summary
Adds a CODEOWNERS entry for /src/content/changelog/security-center/ so that Security Center changelog entries get the same review coverage as the existing /docs/security-center/ and /partials/security-center/ paths.
Why
Currently /src/content/changelog/security-center/ only matches the generic /src/content/changelog/ rule, which owns the changelog dir to @cloudflare/pm-changelogs and @cloudflare/product-owners. That means Security Center–specific reviewers (including the PM for the product) aren't auto-requested when a changelog entry lands under this path — e.g. #31333.
Change
One-line addition under the Security products section, mirroring the existing /docs/security-center/ owners and adding @cloudflare/pm-changelogs (consistent with how other product changelog paths are configured, e.g. /changelog/cloudflare-one/, /changelog/waf/):
/src/content/changelog/security-center/ @alexmoraru7 @cloudflare/appsec-reviewers @elithrar @danielegm @cloudflare/pm-changelogs @cloudflare/product-owners @davejbax @zrkn @Hemanthk1099 @bseel-cfone
Related
Screenshots (optional)
Documentation checklist