Problem/Motivation

The MAINTAINERS.txt file for D7 hasn't been updated for years and lists people who are no longer involved. This gives a false sense of who is currently involved in working on D7 core and who could provide advice on individual topics as they arise.

Proposed resolution

webchick's wording from #3088932: [META] Drupal 7 needs additional maintenance support:

Update the maintainers list so it's accurate. For example, I am still on it, despite not committing anything to Drupal 7 since like 2012. :) There are other folks on there as well who've since left the project, so we should make the list up-to-date.

The branch maintainers are currently listed as:

Remaining tasks

1. Check with the folks above and see whether they still wish to be listed as a core maintainer.
2. Update the file accordingly.

CommentFileSizeAuthor
#21 drupal-n3088938-21.patch736 bytesDamienMcKenna
#21 drupal-n3088938-21.interdiff.txt446 bytesDamienMcKenna
#19 3088938-19.patch734 bytesmcdruid
#12 3088938-remove-inactive-maintainers.patch664 byteswebchick
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DamienMcKenna created an issue. See original summary.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Issue summary: View changes
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented

The branch maintainers are listed as:

Should we get confirmation from these folk if they're all still ready and able to maintain D7?

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented

The subsystem, etc maintainers are

moved to new issue: #3088999: Update the D7 subsystem maintainer list

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented

(I updated comment #4 so the subsystems, etc were grouped by person, rather than the people grouped by subsystem)

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick at Acquia commented

I wonder if we could split off the subsystem maintainer part into its own issue. (And maybe cover it for both branches?) That's a heaping pile more work than contacting 4-5 people.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented

I opened #3088999: Update the D7 subsystem maintainer list for the subsystem maintainers list, so this one can focus on just the primary maintainers.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Issue summary: View changes
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Issue summary: View changes
BrankoC’s picture

BrankoC CreditAttribution: BrankoC as a volunteer commented
Related issues: +#2959509: Remove David Rothstein as Drupal 7 core maintainer
Fabianx’s picture

Fabianx CreditAttribution: Fabianx at Tag1 Consulting commented

I am still active and Pol is active as well.

I haven’t seen David nor Stefan nor webchick nor Dries in quite some time in the D7 queues.

So an update would leave me and Pol.

Can someone write a patch for that scenario?

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick at Acquia commented
Status: Active » Needs review
FileSize
3088938-remove-inactive-maintainers.patch664 bytes

I think we can't remove Dries, since he's still ultimately in charge of the team, but here's a patch for the rest.

BrankoC’s picture

BrankoC CreditAttribution: BrankoC as a volunteer commented

I have asked Dries Buytaert and Stefan Ruijsenaars to state their preference in this issue.

I assume that since Webchick and David Rothstein have brought up this issue respectively wrote a patch for their own removal, they wish to be removed from the list, so I haven't asked them.

(Note: I hadn't seen #12 yet when I sent my messages.)

Dries’s picture

Dries CreditAttribution: Dries commented

I'd like to stay in the list, please. I agree that removing Stefan and David make sense given that they have not been active for some time. I propose we make these changes on November 15th (in ~3 weeks), unless we heard from them, and they are willing to reengage. In the mean time, let's focus on who to add to the Drupal 7 committer team.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick at Acquia commented

David already has a patch for himself over at #2959509: Remove David Rothstein as Drupal 7 core maintainer, so I think we are good there. I guess this is implied by me re-rolling the patch, but I also am fine with being removed; it accurately reflects my activity which is basically solely on D8 (and D9) these days.

BrankoC’s picture

BrankoC CreditAttribution: BrankoC as a volunteer commented

Four weeks ago I asked Dries Buytaert and Stefan Ruijsenaars to respond in this issue. Dries responded in #14. I haven't heard from Stefan Ruijsenaars; not in this issue, nor otherwise.

webchick’s picture

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

Yep, given that, I think this is good to go as-is, as it removes myself, David Rothstein (who already asked to be removed), and Stefan (who has not been responsive).

mcdruid’s picture

mcdruid
Location 🇬🇧🇪🇺
CreditAttribution: mcdruid at Acquia commented
Status: Reviewed & tested by the community » Needs work
Related issues: +#3089877: Resign from my role of provisional maintainer, +#3088557: Volunteering as a D7 core committer (@mcdruid)

Added a couple more related issues.

I've been added as a provisional maintainer for D7 (which will be why the patch no longer applies; I'll fix that).

Plus @Pol has stepped down (I'd like to get solid confirmation of that decision before actually removing his entry from MAINTAINERS.txt though).

mcdruid’s picture

mcdruid
Location 🇬🇧🇪🇺
CreditAttribution: mcdruid at Acquia commented
Status: Needs work » Needs review
FileSize
3088938-19.patch734 bytes

New patch which removes the same entries.

I'm trying to contact @Pol to confirm what he's decided about #3089877: Resign from my role of provisional maintainer

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick at Acquia commented
Status: Needs review » Needs work

#3089877-13: Resign from my role of provisional maintainer :(

I guess we need Pol's name added to the list of departees.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Needs work » Needs review
FileSize
drupal-n3088938-21.interdiff.txt446 bytes
drupal-n3088938-21.patch736 bytes

Removed Pol.

That said, the other issue be committed and let this issue continue separately?

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick at Acquia commented
Assigned: Unassigned » Dries
Status: Needs review » Reviewed & tested by the community

Could do; but I think trying to apply one before/after the other will cause a conflict.

Assigning to @Dries and marking RTBC.

mcdruid’s picture

mcdruid
Location 🇬🇧🇪🇺
CreditAttribution: mcdruid at Acquia commented
Assigned: Dries » Unassigned

Would like to get this update committed in time for the scheduled release this week if possible :)

mcdruid’s picture

mcdruid
Location 🇬🇧🇪🇺
CreditAttribution: mcdruid at Acquia commented
Assigned: Unassigned » Dries

Oops sorry, didn't mean to un-assign.

Fabianx’s picture

Fabianx CreditAttribution: Fabianx at Tag1 Consulting commented

RTBC + 1

Dries’s picture

Dries CreditAttribution: Dries commented

+1 from me. Feel free to commit this patch.

  • mcdruid committed f5f4841 on 7.x 
    Issue #3088938 by DamienMcKenna, webchick, mcdruid: Update the D7...
mcdruid’s picture

mcdruid
Location 🇬🇧🇪🇺
CreditAttribution: mcdruid at Acquia commented
Assigned: Dries » Unassigned
Status: Reviewed & tested by the community » Fixed

Thank you everyone!

Status: Fixed » Closed (fixed)

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