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:
- Dries Buytaert 'dries' https://www.drupal.org/u/dries
- Angela Byron 'webchick' https://www.drupal.org/u/webchick
- Fabian Franz 'Fabianx' https://www.drupal.org/u/fabianx
- David Rothstein 'David_Rothstein' https://www.drupal.org/u/david_rothstein
- Stefan Ruijsenaars 'stefan.r' https://www.drupal.org/u/stefanr-0
- (provisional) Pol Dellaiera 'Pol' https://www.drupal.org/u/pol
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.
Comment | File | Size | Author |
---|---|---|---|
#21 | drupal-n3088938-21.patch | 736 bytes | DamienMcKenna |
#21 | drupal-n3088938-21.interdiff.txt | 446 bytes | DamienMcKenna |
#19 | 3088938-19.patch | 734 bytes | mcdruid |
#12 | 3088938-remove-inactive-maintainers.patch | 664 bytes | webchick |
Comments
Comment #2
DamienMcKennaComment #3
DamienMcKennaThe branch maintainers are listed as:
Should we get confirmation from these folk if they're all still ready and able to maintain D7?
Comment #4
DamienMcKennaThe subsystem, etc maintainers are
moved to new issue: #3088999: Update the D7 subsystem maintainer list
Comment #5
DamienMcKenna(I updated comment #4 so the subsystems, etc were grouped by person, rather than the people grouped by subsystem)
Comment #6
webchickI 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.
Comment #7
DamienMcKennaI opened #3088999: Update the D7 subsystem maintainer list for the subsystem maintainers list, so this one can focus on just the primary maintainers.
Comment #8
DamienMcKennaComment #9
DamienMcKennaComment #10
BrankoC CreditAttribution: BrankoC as a volunteer commentedComment #11
Fabianx CreditAttribution: Fabianx at Tag1 Consulting commentedI 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?
Comment #12
webchickI think we can't remove Dries, since he's still ultimately in charge of the team, but here's a patch for the rest.
Comment #13
BrankoC CreditAttribution: BrankoC as a volunteer commentedI 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.)
Comment #14
Dries CreditAttribution: Dries commentedI'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.
Comment #15
webchickDavid 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.
Comment #16
BrankoC CreditAttribution: BrankoC as a volunteer commentedFour 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.
Comment #17
webchickYep, 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).
Comment #18
mcdruidAdded 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).
Comment #19
mcdruidNew 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
Comment #20
webchick#3089877-13: Resign from my role of provisional maintainer :(
I guess we need Pol's name added to the list of departees.
Comment #21
DamienMcKennaRemoved Pol.
That said, the other issue be committed and let this issue continue separately?
Comment #22
webchickCould do; but I think trying to apply one before/after the other will cause a conflict.
Assigning to @Dries and marking RTBC.
Comment #23
mcdruidWould like to get this update committed in time for the scheduled release this week if possible :)
Comment #24
mcdruidOops sorry, didn't mean to un-assign.
Comment #25
Fabianx CreditAttribution: Fabianx at Tag1 Consulting commentedRTBC + 1
Comment #26
Dries CreditAttribution: Dries commented+1 from me. Feel free to commit this patch.
Comment #28
mcdruidThank you everyone!