Problem/Motivation

workflows.module is a fully fledged module in its own right
It should have a component in drupal.org issue queue and an entry in MAINTAINERS.txt

Proposed resolution

Agree.
Add component to issue queue.
Add maintainer to MAINTAINERS.txt
I would like to nominate Sam152 as maintainer.
Looking at the list of issues he has contributed to in workflow in the last three months I feel this is an appropriate choice - assuming he is willing to take on the role.
In addition he has spent tireless hours working on #2846618: Unofficial content_moderation 8.2.x to 8.3.0 upgrade path and #2890187: [META] Unofficial content_moderation upgrade path - both as patches and providing support which will never make it into core and get the appropriate recognition.

Remaining tasks

Agree.
Add component to issue queue.
Add maintainer to MAINTAINERS.txt

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#10 2896063-4.patch392 bytesxjm
#9 2896063-9.patch839 bytessam152
#4 2896063-4.patch392 bytestimmillwood

Comments

larowlan created an issue. See original summary.

larowlan’s picture

Title: Add a workflow component to drupal issue queue » Add a workflows.module component to drupal issue queue and MAINTAINERS.txt
Issue summary: View changes
larowlan’s picture

Issue summary: View changes
timmillwood’s picture

Status: Active » Needs review
Issue tags: +Workflow Initiative
StatusFileSize
new392 bytes

I'm a strong supporter of each module having a listed maintainer so new and old core contributors have a point of contact. Sam has been an awesome supporter and contributor to Workflows module, before and after it was split from Content Moderation, so I support @larowlan's nomination.

timmillwood’s picture

Edit: 500 error caused double post.

sam152’s picture

Status: Needs review » Reviewed & tested by the community

Appreciate the vote of confidence from @larowlan and @timmillwood. It'll be great to start moving the workflows issues off into their own queue, once the component is created.

I have read and agree to the responsibilities listed here.

gábor hojtsy’s picture

Looks good to me, thanks for stepping up! :) I know @xjm and @catch had reservations before about expanding the components list so I pinged @catch about this.

xjm’s picture

Status: Reviewed & tested by the community » Needs review
Issue tags: +Component maintainers
Related issues: +#2660144: [Plan] Update core components

Thanks @Sam152!

If it's alright, I'd rather have a single "Workflows and Content Moderation" component or such for Workflow, Content Moderation, etc. We're trying to reduce the number of options in the select list and group related maintainers in the file. "Views" is now a single entry, for example, even though views_ui.module is separate. If the maintainers are not the same for the individual submodules, then I think we should group them. See the database subsystem or PHPUnit initiative for examples.

sam152’s picture

StatusFileSize
new839 bytes

Thanks @Gábor Hojtsy and @xjm!

The hope is workflows will be useful beyond content moderation, in contrib and core, possibly beyond the workflow initiative. From the perspective of managing the queue, I think it'd be helpful to have a seperate component. I did experiment with using a tag for this, but it didn't quite work, as you can see :)

If that's simply not feasible, I'd suggest a rename to something like "Workflow Initiative", we'll hit the same problem once workspaces are introduced.

As far as the layout of MAINTAINERS.txt, can't see a problem with grouping them together. Attached patch matches the format of the database API.

xjm’s picture

Status: Needs review » Reviewed & tested by the community
StatusFileSize
new392 bytes

The hope is workflows will be useful beyond content moderation, in contrib and core, possibly beyond the workflow initiative. From the perspective of managing the queue, I think it'd be helpful to have a seperate component. I did experiment with using a tag for this, but it didn't quite work, as you can see :)

Thanks @Sam152. Okay, this convinced me. :) At least while #2702321: Group core components into broad categories and #2449085: Make "Component" a multi-value field, or allow sub-components are outstanding, I guess we should add yet-another-component. (Especially since you found the tagging to be ineffective.)

Initiative coordinators have very different responsibilities than subsystem maintainers (compare https://www.drupal.org/contribute/core/maintainers#subsystem and https://www.drupal.org/contribute/core/maintainers#initiative) and Initiative Coordinators also need to be approved by Dries. So #4 is probably the correct patch. So reuploading the earlier patch and setting back to RTBC.

I'll leave this RTBC a day or so before committing since I'm the one who bounced it and then changed my mind.

timmillwood’s picture

Another idea is list all the people heavily working on the initiative, then under each module list the initiative team as the maintainer.

But I am still OK with the patch in #4 / #10.

xjm’s picture

That could work for an initiative like VDC or WI, but not for one like Multilingual. :) Initiatives also eventually end and are removed from the file when they complete their goals, while the modules stick around and can have new active maintainers. Some initiatives also don't add modules really.

Thanks @timmillwood!

gábor hojtsy’s picture

Agreed with adding new component and committing this maintainer addition.

  • xjm committed 8176f53 on 8.4.x
    Issue #2896063 by timmillwood, Sam152: Add a workflows.module component...

  • xjm committed 550cff1 on 8.3.x
    Issue #2896063 by timmillwood, Sam152: Add a workflows.module component...
xjm’s picture

Version: 8.4.x-dev » 8.3.x-dev
Status: Reviewed & tested by the community » Fixed

Okay, committed and pushed to 8.4.x and 8.3.x. Thanks so much @Sam152 for volunteering for this role!

I've added the workflows.module component. @Sam152, I've also granted you "Maintain issues" permission (enjoy the sprawling "Assigned" field on core issues!) and added you as a normal member of https://groups.drupal.org/core to post announcements about your subsystem as needed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.