Skip to content

Add blanket docs-team CODEOWNERS rule (DOC-2496) - #430

Open
Feediver1 wants to merge 1 commit into
mainfrom
doc-2496-codeowners
Open

Add blanket docs-team CODEOWNERS rule (DOC-2496)#430
Feediver1 wants to merge 1 commit into
mainfrom
doc-2496-codeowners

Conversation

@Feediver1

@Feediver1 Feediver1 commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

What

Adds the blanket docs-team CODEOWNERS rule (* @redpanda-data/documentation) for DOC-2496, the on-call docs PR review pipeline. This file is inert until branch protection enables "require review from Code Owners" — that flip is a separate, announced step (repo admins; see the rollout guide in docs-team-standards .github/DOCS_REVIEW_ROLLOUT.md).

Once protection is enabled, a docs-team approval becomes the merge gate here — that's the pipeline's merge-authority model (design decision 7: the automated docs-review status stays informational; only the CODEOWNERS approval physically gates).

Part of DOC-2496; pipeline PRs: redpanda-data/docs-team-standards#2912, redpanda-data/docs-extensions-and-macros#303.

🤖 Generated with Claude Code

Inert until branch protection requires code-owner review; that flip is a
separate announced step. Part of the on-call docs PR review pipeline
(DOC-2496).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@netlify

netlify Bot commented Sep 3, 2026

Copy link
Copy Markdown

Deploy Preview for docs-ui ready!

Name Link
🔨 Latest commit 8bbf65d
🔍 Latest deploy log https://app.netlify.com/projects/docs-ui/deploys/6a99b27e63f40d000850eab4
😎 Deploy Preview https://deploy-preview-430--docs-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 28 (🔴 down 3 from production)
Accessibility: 89 (no change from production)
Best Practices: 92 (no change from production)
SEO: 89 (no change from production)
PWA: -
View the detailed breakdown and full score reports
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai

coderabbitai Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Essentials

Run ID: fa3d878b-1371-4276-b678-93e5cb27683d

📥 Commits

Reviewing files that changed from the base of the PR and between 97eceb0 and 8bbf65d.

📒 Files selected for processing (1)
  • .github/CODEOWNERS

Included review availability: 3 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.


📝 Walkthrough

Walkthrough

The CODEOWNERS file now includes a global * rule that assigns all paths to the documentation team. Comments explain the rule's branch-protection purpose. Existing path-specific rules retain precedence.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 8bbf6

This change assigns the documentation team as the default code owner while preserving more specific ownership rules. No current merge-readiness risk remains.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: adding a blanket documentation-team CODEOWNERS rule.
Description check ✅ Passed The description directly explains the CODEOWNERS change, its relationship to DOC-2496, and its branch-protection dependency.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch doc-2496-codeowners

Comment @coderabbitai help to get the list of available commands.

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