
Problem/Motivation
In #3439923: Add recipes api as experimental API to core we added 4 recipes APIs to core. As a follow up, we need to identify and add subsystems on Drupal.org and subsystem maintainers in MAINTAINERS.txt. These maintainers have special roles in the governance structure and additional permissions, like the ability to adjust the Assigned field on behalf of others, assign issue credits in core, etc. See https://www.drupal.org/contribute/core/maintainers#subsystem for what this means.
The new APIs added were:
- Drupal\Core\Config\Action
- Drupal\Core\Config\Checkpoint
- Drupal\Core\Recipe
- Drupal\Core\DefaultContent
In this conversation in the Recipes channel of Drupal Slack, @alexpott determined that we need two subsystems, ‘recipe system’ and ‘default content system’. The Config Actions and Config Checkpoint issues can live in existing systems.
I think config actions can stay in config or config entity system… but I think we need ‘recipe system’ and ‘default content system’
Proposed resolution
- Add ‘recipe system’ and ‘default content system’ subsystems to Drupal.org.
- Identify the subsystem maintainers.
- Provide a patch to MAINTAINERS.txt (committers will do the followup tasks of assigning maintainer permissions, etc).
Remaining tasks
Do it.
User interface changes
API changes
Data model changes
Release notes snippet
Issue fork drupal-3447072
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #2
thejimbirch CreditAttribution: thejimbirch at Kanopi Studios commentedComment #3
thejimbirch CreditAttribution: thejimbirch at Kanopi Studios commentedIn the same slack thread, @alexpott identified the maintainers as
@alexpott
@phenaproxima
@thejimbirch
Comment #5
thejimbirch CreditAttribution: thejimbirch at Kanopi Studios commentedI read the Subsystem maintainer responsibilities and general maintainer responsibilities and agree.
Comment #6
alexpott@thejimbirch thanks for working on this. I'm happy to be in MAINTAINERS.txt for default content and recipes.
Comment #9
alexpottComment #10
dwwLooks good to me, though we probably want @phenaproxima to formally accept in here before this is committed.
Comment #11
phenaproximaI formally accept co-maintainership of the recipe subsystem.
Comment #12
dwwGroovy, thanks. Ship it! 🎉
Comment #13
longwaveThanks for volunteering to maintain these new subsystems! I've added the two new subsystems to the component dropdown.
Committed and pushed a885cc86c5 to 11.x and 7df7eb4c51 to 11.0.x and 8a26f33828 to 10.4.x and 47794d4077 to 10.3.x. Thanks!
Comment #18
longwaveComment #19
longwave