This is not about finding anyone to fill this in, that can be handled separately. This doesn't require any specific people to chime in, it just rounds out the file to include an entry for each.

CommentFileSizeAuthor
module-maintainers.patch1.28 KBtim.plunkett
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented

I checked this patch vs. the actual modules at core/modules (and subdirectories such as Custom Block being the only one that isn't in its own -- why is that anyway??? Wait, don't answer that here).

The patch correctly adds all the missing modules.

Question: Is it out of scope to ask that the module names in MAINTAINERS match the actual module names, and that they be ordered alphabetically?

Also, is MAINTAINERS supposed to list modules by their machine names or human-readable names from the info file? It's a mix. Would fixing that be out of scope?

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
CreditAttribution: tim.plunkett commented

Help is out of order. Phone module is telephone.module. Many use their human name and are out of the module order (hal -> hypertext).

None of that is in scope here :)

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented
Component: other » documentation
Status: Needs review » Reviewed & tested by the community

Given the scope, this patch is good.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, module-maintainers.patch, failed testing.

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
CreditAttribution: tim.plunkett commented
Status: Needs work » Reviewed & tested by the community

LOL at Drupal\user\Tests\UserCancelTest

larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
CreditAttribution: larowlan commented

for custom block see #1920862: Rename custom_block.module to block_content.module

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Ignoring testbot's opinion of this patch. :P

Committed and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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