Skip to content

Add/commit access docs#857

Merged
dkotter merged 3 commits into
developfrom
add/commit-access-docs
Jul 13, 2026
Merged

Add/commit access docs#857
dkotter merged 3 commits into
developfrom
add/commit-access-docs

Conversation

@jeffpaul

@jeffpaul jeffpaul commented Jul 10, 2026

Copy link
Copy Markdown
Member

What?

This PR adds a CODEOWNERS file and docs on how to get added as a Codeowner and then as a Maintainer.

Copilot summary

This pull request updates project documentation and repository ownership to clarify the code review and merging process, and to ensure proper assignment of code owners for different parts of the repository. The most important changes are:

Repository Ownership and Code Review Process:

  • .github/CODEOWNERS: Added a global fallback code owner (@WordPress/ai-maintainers), specified ownership for documentation and project management directories, and clarified who is responsible for code reviews.

Documentation Updates:

  • docs/DEVELOPER_GUIDE.md: Added detailed sections describing the PR merging process, steps for joining the ai-maintainers team, and guidelines for closing PRs, including communication best practices and a template for closing PRs.

Why?

How?

Use of AI Tools

AI assistance: none besides Copilot in the PR description

Testing Instructions

Screenshots or screencast

Before After

Changelog Entry

Developer - Added docs on becoming a Maintainer.

Open WordPress Playground Preview

@jeffpaul jeffpaul added this to the 1.2.0 milestone Jul 10, 2026
@jeffpaul
jeffpaul requested a review from dkotter July 10, 2026 21:31
@jeffpaul jeffpaul self-assigned this Jul 10, 2026
@jeffpaul
jeffpaul requested a review from a team July 10, 2026 21:31
@github-actions

github-actions Bot commented Jul 10, 2026

Copy link
Copy Markdown

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: jeffpaul <jeffpaul@git.wordpress.org>
Co-authored-by: dkotter <dkotter@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@jeffpaul

Copy link
Copy Markdown
Member Author

@dkotter I tried to write up some notes on getting folks elevated to Maintainer after reading through how Gutenberg handles things and trying to get some specifics into our docs to help quantify/qualify the process. Curious about your input here and whether I've over- or under-described things?

@jeffpaul jeffpaul moved this from Triage to Needs review in WordPress AI Roadmap Jul 10, 2026
@github-actions

Copy link
Copy Markdown

✅ WordPress Plugin Check Report

✅ Status: Passed

📊 Report

All checks passed! No errors or warnings found.


🤖 Generated by WordPress Plugin Check Action • Learn more about Plugin Check

@codecov

codecov Bot commented Jul 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.05%. Comparing base (8303620) to head (b1917fa).

Additional details and impacted files
@@            Coverage Diff             @@
##             develop     #857   +/-   ##
==========================================
  Coverage      77.05%   77.05%           
  Complexity      2233     2233           
==========================================
  Files            103      103           
  Lines           9216     9216           
==========================================
  Hits            7101     7101           
  Misses          2115     2115           
Flag Coverage Δ
unit 77.05% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jeffpaul jeffpaul mentioned this pull request Jul 13, 2026
28 tasks
@dkotter

dkotter commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

@dkotter I tried to write up some notes on getting folks elevated to Maintainer after reading through how Gutenberg handles things and trying to get some specifics into our docs to help quantify/qualify the process. Curious about your input here and whether I've over- or under-described things?

This looks good to me. Can adjust as needed but this is a good place to start

@dkotter
dkotter merged commit 8c3606a into develop Jul 13, 2026
24 checks passed
@dkotter
dkotter deleted the add/commit-access-docs branch July 13, 2026 22:52
@github-project-automation github-project-automation Bot moved this from Needs review to Done in WordPress AI Roadmap Jul 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants