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:
- Lucas Hedding 'heddn' https://www.drupal.org/u/heddn
- Valery Lourie 'valthebald' https://www.drupal.org/u/valthebald
- Alina Mackenzie 'alimac' https://www.drupal.org/u/alimac
- Jess Myrbo 'xjm' https://www.drupal.org/u/xjm
- Andrea Soper 'ZenDoodles' https://www.drupal.org/u/zendoodles
- Cathy Theys 'YesCT' https://www.drupal.org/u/yesct
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
- Ask current maintainers what is their actual status with the core mentoring program
- Create patch
- Update past maintainers page
User interface changes
None
API changes
None
Data model changes
None
| Comment | File | Size | Author |
|---|---|---|---|
| #20 | interdiff-17-20.txt | 760 bytes | mradcliffe |
| #20 | 2900394-20.patch | 947 bytes | mradcliffe |
| #17 | 2900394-17.patch | 996 bytes | heddn |
Comments
Comment #2
valthebaldResponses so far from:
valthebald - keep active
alimac - step down
xjm - keep active
YesCT - step down
Comment #3
valthebaldComment #4
rachel_norfolkI'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.
Comment #5
rachel_norfolkAlso 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".
Comment #6
heddnLooks like the wrong patch got added here?
Comment #7
heddnI can still help mentor at DC north america.
Comment #8
rachel_norfolkOh - 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
Comment #9
dinarcon commentedChris 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
Comment #10
valthebald@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.
Comment #11
cilefen commentedVal: your memory is correct.
Comment #12
valthebaldRemoving Andrea from the list (but worth letting more time for the feedback from her side)
Comment #13
valthebaldAndrea has confirmed that we can remove her from the current maintainers list. RTBC anyone?
Comment #14
rachel_norfolkJust adding that thing you forgot, Val ;-)
Yes, this is simple and we have replies from all those involved. Thanks for calling round!
Comment #15
heddn+1 on RTBC.
Comment #16
cilefen commentedI 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.
Comment #17
heddnAlpha 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.
Comment #18
xjmTypically it's alphabetical by last name. :) E.g. see the committers section.
Comment #19
xjmLet's also add those stepping down to the past maintainers page before commit. Thanks!
Comment #20
mradcliffeHere is an updated patch with last names in alphabetical order.
I believe I can commit to helping organize mentoring event through 2018 at least.
Comment #21
heddnI've updated https://www.drupal.org/node/2794489 as well. So all actionable steps are now completed. Back to RTBC.
Comment #23
xjmThanks @mradcliffe, @dinarcon, and @rachel_norfolk! Committed to 8.5.x and cherry-picked to 8.4.x.
Comment #25
xjm(Testing the issue crediting for committers.)
Comment #26
xjm