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

Comments

jhodgdon’s picture

jhodgdon
she/her
Primary language English
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
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
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
commented
Status: Needs work » Reviewed & tested by the community

LOL at Drupal\user\Tests\UserCancelTest

larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
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 🇨🇦
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.