Problem/Motivation

@mcdruid has been a provisional maintainer of the 7.x branch for approaching a year (commit).

In that time, he's contributed to and overseen 6 releases (7.68 - 7.73) and published and maintained a Roadmap / Release Schedule for D7.

To quote myself (Fabianx):

I'd like mcdruid to be a real committer (no longer provisional) as he does great work and presents patches that are for most cases directly committable and don't need further work. In addition for complex patches he always solicits feedback from me and others.

Therefore I am very happy to propose that mcdruid be made a full core committer.

Proposed resolution

Make @mcdruid a full maintainer of the 7.x branch.

Remaining tasks

Review and commit patch to MAINTAINERS.txt

CommentFileSizeAuthor
#4 3174366-4.patch461 bytesmcdruid
#2 3174366-2.patch19.22 KBmcdruid

Comments

mcdruid created an issue. See original summary.

mcdruid’s picture

Status: Active » Needs review
StatusFileSize
new19.22 KB
mcdruid’s picture

Title: Promote mcdruid to full maintainer of D7 » Promote mcdruid to full D7 maintainer
mcdruid’s picture

StatusFileSize
new461 bytes

Oops, and here's the patch without some other stuff I was working on.

longwave’s picture

I don't know who gets to RTBC this but +1 from me.

fabianx’s picture

Issue summary: View changes
fabianx’s picture

Assigned: Unassigned » dries

I'd like mcdruid to be a real committer (no longer provisional) as he does great work and presents patches that are for most cases directly committable and don't need further work. In addition for complex patches he always solicits feedback from me and others.

Therefore I am very happy to propose that mcdruid be made a full core committer.

--

Therefore:

The patch is RTBC, but now needs to be approved by Dries, who puts the status to RTBC.

Assigning to Dries

webchick’s picture

I don't think you need another +1 here, but +1 anyway. :) As the original release manager of Drupal 7, I've been repeatedly impressed with @mcdruid's ability to prioritize critical improvements for D7, to exercise care and caution in testing prior to merging in changes, and his encouragement of other contributors. Glad you'll be joining the team as a full member! :)

joseph.olstad’s picture

+1
Another plug for
#973436: Overzealous locking in variable_initialize()
I have mentioned this issue to McDruid, Imploring this rtbc to go in asap , several of my clients have needed this and it has cost my clients a lot of money over the years (to pay me for debugging and critical urgent work as they discovered unexpected consequences related to not having this 10 yr old rtbc fix)

McDruid doing great work

joseph.olstad’s picture

Status: Needs review » Reviewed & tested by the community

Any objections?

dries’s picture

No objections. Approved! Thanks for all the hard work.

gábor hojtsy’s picture

Assigned: dries » Unassigned

  • Fabianx committed 997f000 on 7.x
    Issue #3174366 by Fabianx, Dries: Promote mcdruid to full D7 maintainer
    
fabianx’s picture

Status: Reviewed & tested by the community » Fixed

Fixed, thanks for the patience!

Status: Fixed » Closed (fixed)

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