Add initial code owners configuration
Moving checks into separate folders per category makes it easy to set up code owners.
Feel free to submit suggestions to the CODEOWNERS file if you like to be included/excluded.
See #448
We can create new teams for each category or something if we want, e.g. @WordPress/plugin-check-plugins-team, @WordPress/plugin-check-performance. Don't want to use that existing team though, as it includes too many people who aren't contributing this plugin and would just unnecessarily get pinged because of it (like right now :)).
(Edit: also keep in mind mentioning/assigning teams doesn't work for people not in the org or when logged out of GitHub)
Not worried about back compat, a changelog entry should be enough.
Who is responsible for the other checks and other infrastructure-related files?
Everyone and no one :) Something we can define as we go. Suggestions welcome if you think we should define all of that already.
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: swissspidy <[email protected]>
Co-authored-by: joemcgill <[email protected]>
Co-authored-by: westonruter <[email protected]>
Co-authored-by: mukeshpanchal27 <[email protected]>
Co-authored-by: davidperezgar <[email protected]>
Co-authored-by: adamsilverstein <[email protected]>
To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.
Looks great and this will let individual teams iterate more rapidly. +1.
@bordoni @chriscct7 @davidperezgar @barrykooij Would appreciate eyes on this from the plugins team as well as it means you'll get notified more often for PRs (every time some touches checks related to your team).
IMHO adding a code owners configuration is a great way to more clearly separate concerns. Even if we end up restructuring the plugin further down the road, this would be a good thing to do.
I wait @barrykooij have an opinion of this.
But it looks great as we have very clear information where to see the checks.