Add explicit CODEOWNERS rule for /src/content/partials/load-balancing/#30975
Conversation
Mirror the /src/content/docs/load-balancing/ rule for /src/content/partials/load-balancing/ so PRs touching load-balancing partials no longer fall through to the default '* @cloudflare/product-owners' rule. This is consistent with how other products handle their partials (e.g., /src/content/partials/r2/, /src/content/partials/d1/, /src/content/partials/workers/ all have explicit rules) and lets the same set of reviewers approve docs and partials together.
|
This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:
|
|
No issues found. The new rule mirrors the existing docs ownership, is placed correctly within the Performance products section, and matches the pattern used by other products. |
|
PR #30975 review complete. Labels applied: Summary: Issues flagged: None. The rule is syntactically correct, placed immediately after the corresponding Verdict: Ready to merge. |
|
Preview URL: https://e461ec74.preview.developers.cloudflare.com |
Summary
Adds an explicit CODEOWNERS rule for
/src/content/partials/load-balancing/mirroring the owners of/src/content/docs/load-balancing/:Why
Today, load-balancing partials fall through to the top-level default rule (
* @cloudflare/product-owners), which means PRs that touch bothdocs/load-balancing/andpartials/load-balancing/require approvals from two different sets of reviewers — even though the same Load Balancing team owns both. For example, #30938 touches both, and my approval as a named code owner on/docs/load-balancing/does not satisfy the default rule that covers the partials file.This is consistent with how other products handle their partials —
/src/content/partials/r2/,/src/content/partials/d1/,/src/content/partials/workers/,/src/content/partials/hyperdrive/, etc. all have explicit rules that match (or extend) their correspondingdocs/rule.Documentation checklist