Skip to content

[Browser Run] Add engineering team as code owners for docs#30942

Merged
kathayl merged 3 commits into
productionfrom
add-br-team-codeowners
May 20, 2026
Merged

[Browser Run] Add engineering team as code owners for docs#30942
kathayl merged 3 commits into
productionfrom
add-br-team-codeowners

Conversation

@kathayl
Copy link
Copy Markdown
Contributor

@kathayl kathayl commented May 19, 2026

Summary

Adds Browser Run engineering team members as code owners for /src/content/docs/browser-run/ so they can review and approve docs changes directly.

New code owners added:

  • @meddulla (Sofia Cardita)
  • @simonabadoiu (Simona Badoiu)
  • @jonnyparris (Ruskin Constant)
  • @ruifigueira (Rui Figueira)

These team members already have GitHub seats in the Cloudflare org.

@kathayl kathayl requested review from a team and kodster28 as code owners May 19, 2026 22:05
@github-actions
Copy link
Copy Markdown
Contributor

This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:

Pattern Owners
/.github/CODEOWNERS @cloudflare/product-owners, @cloudflare/content-engineering, @kodster28

Copy link
Copy Markdown
Contributor

@ask-bonk ask-bonk Bot left a comment

Choose a reason for hiding this comment

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

Clean, straightforward change. The four new code owners are correctly added to the Browser Run docs path.

Suggestion (non-blocking): Consider also adding the team to the related Browser Run content directories so they can review all product-specific docs changes:

  • (10 partials)
  • (15+ changelog entries)

Many other product areas include these related paths (see ai-crawl-control, agents, d1, etc.).

@ask-bonk
Copy link
Copy Markdown
Contributor

ask-bonk Bot commented May 19, 2026

The file paths were stripped from my earlier review comment. Here is the complete suggestion:

Consider also adding the team to these related Browser Run content directories:

  • src/content/partials/browser-run/ (10 partials)
  • src/content/changelog/browser-run/ (15+ changelog entries)
  • src/content/release-notes/browser-run.yaml
  • src/assets/images/browser-run/

Many other product areas include these related paths so the engineering team can review all product-specific content.

@ask-bonk
Copy link
Copy Markdown
Contributor

ask-bonk Bot commented May 19, 2026

Done. Here's my review summary for PR #30942:

Labels applied: product:browser-run

Review outcome: Approved

Summary: The PR correctly adds four Browser Run engineering team members (@meddulla, @simonabadoiu, @jonnyparris, @ruifigueira) as code owners for /src/content/docs/browser-run/. The usernames are formatted correctly and the change aligns with the existing CODEOWNERS pattern.

Flagged issue (non-blocking): I suggested considering coverage for the related Browser Run content directories that currently have no product-specific owners:

  • src/content/partials/browser-run/ (10 partials)
  • src/content/changelog/browser-run/ (15+ changelog entries)
  • src/content/release-notes/browser-run.yaml
  • src/assets/images/browser-run/

Many other product areas (e.g., ai-crawl-control, agents, d1) include these adjacent paths so their engineering teams can review all product-specific docs changes. If the intent is full docs ownership, this would complete the coverage.

github run

@kathayl
Copy link
Copy Markdown
Contributor Author

kathayl commented May 19, 2026

/bonk proceed with your recommendation to include adjacent paths

Co-authored-by: kathayl <kathayl@users.noreply.github.com>
@ask-bonk
Copy link
Copy Markdown
Contributor

ask-bonk Bot commented May 19, 2026

Done. I've updated .github/CODEOWNERS to add the Browser Run team to the adjacent content directories:

  • src/content/partials/browser-run/
  • src/content/changelog/browser-run/
  • src/content/release-notes/browser-run.yaml
  • src/assets/images/browser-run/

All four paths use the same owner list as the existing /src/content/docs/browser-run/ entry.

github run

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 19, 2026

@kathayl kathayl merged commit f0adf72 into production May 20, 2026
19 checks passed
@kathayl kathayl deleted the add-br-team-codeowners branch May 20, 2026 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants