Problem/Motivation

It has been 2 years since last review of mentoring coordinators section of MAINTAINERS.txt by #2522716: Update mentoring description in MAINTAINERS.txt and add current maintainers alimac, heddn, valthebald, cilefen. Since this review, some people have defacto stepped down, and some have stepped up to lead the mentoring efforts.

Proposed resolution

1. Ask all people currently listed as core mentoring coordinators:

2. Move people who have stepped down, to the past maintainers page

3. Remove people who have stepped down from MAINTAINERS.txt

4. Add new de-facto coordinators: Rachel Lawson, Mauricio Dinarte and Matthew Radcliffe (did I miss someone? please add) to

Remaining tasks

  1. Ask current maintainers what is their actual status with the core mentoring program
  2. Create patch
  3. Update past maintainers page

User interface changes

None

API changes

None

Data model changes

None

Members fund testing for the Drupal project. Drupal Association Learn more

Comments

valthebald created an issue. See original summary.

valthebald’s picture

Responses so far from:

valthebald - keep active
alimac - step down
xjm - keep active
YesCT - step down

valthebald’s picture

Issue summary: View changes
rachel_norfolk’s picture

I'm happy to be added so making a patch assuming others are too and, once they confirm, we can continue. Easy to change if not.

rachel_norfolk’s picture

Version: 8.5.x-dev » 8.4.x-dev

Also seems appropriate to set to 8.4.x as otherwise will be 7 months out of date by the time it arrives!

We are in alpha for 8.4 but there are no specific restrictions on what changes are allowed in alpha releases beyond the allowed changes for minors. I can't see this as "disruptive".

heddn’s picture

Looks like the wrong patch got added here?

heddn’s picture

I can still help mentor at DC north america.

rachel_norfolk’s picture

Status: Active » Needs review
FileSize
801 bytes

Oh - I’m so sorry!! No idea why it came up first in my list!

Attached correct patch, I hope!!

Decided to add tests to at least demonstrate the patch applies

dinarcon’s picture

Chris McCafferty 'cilefen' https://www.drupal.org/u/cilefen has mentored in the last North American DrupalCons and participated in some of the monthly meetings. It would be good to check with him whether he wants to be added to the list of current mentor coordinators. Note that he is already listed in as past mentor coordinator https://www.drupal.org/core/maintainers/past

valthebald’s picture

@dinarcon if I remember correctly, Chris has asked to be removed from the mentoring coordinators section by himself, as he is focused on other core tasks. But you're right, worth double-checking.

cilefen’s picture

Val: your memory is correct.

valthebald’s picture

Removing Andrea from the list (but worth letting more time for the feedback from her side)

valthebald’s picture

Andrea has confirmed that we can remove her from the current maintainers list. RTBC anyone?

rachel_norfolk’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
638 bytes

Just adding that thing you forgot, Val ;-)

Yes, this is simple and we have replies from all those involved. Thanks for calling round!

heddn’s picture

+1 on RTBC.

cilefen’s picture

I hate to be a nudge, but hasn't this section been in alphabetical order to date? Some other sections of this file are in seniority, or in lead/team order.

heddn’s picture

Status: Reviewed & tested by the community » Needs review
FileSize
996 bytes

Alpha order. There isn't really a leader. No interdiff, as it would be meaningless. And we should get a comment from mradcliffe in here too, as we don't have any acceptance from him that he is willing to fulfill the responsibilities of maintainership.

Back to NR waiting on a confirmation from Matt and a review of the new order.

xjm’s picture

Status: Needs review » Needs work

Typically it's alphabetical by last name. :) E.g. see the committers section.

xjm’s picture

Let's also add those stepping down to the past maintainers page before commit. Thanks!

mradcliffe’s picture

Status: Needs work » Needs review
FileSize
947 bytes
760 bytes

Here is an updated patch with last names in alphabetical order.

I believe I can commit to helping organize mentoring event through 2018 at least.

heddn’s picture

Status: Needs review » Reviewed & tested by the community

I've updated https://www.drupal.org/node/2794489 as well. So all actionable steps are now completed. Back to RTBC.

  • xjm committed 2059dc9 on 8.5.x
    Issue #2900394 by rachel_norfolk, mradcliffe, valthebald, heddn: Update...
xjm’s picture

Status: Reviewed & tested by the community » Fixed

Thanks @mradcliffe, @dinarcon, and @rachel_norfolk! Committed to 8.5.x and cherry-picked to 8.4.x.

  • xjm committed bfed6f8 on 8.4.x
    Issue #2900394 by rachel_norfolk, mradcliffe, valthebald, heddn: Update...
xjm’s picture

(Testing the issue crediting for committers.)

xjm’s picture

Issue tags: +MWDS2017

Status: Fixed » Closed (fixed)

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