Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.As mentioned in https://www.drupal.org/project/drupal/issues/3187241#comment-13980588, there are currently Provisional memberships on the Core mentoring coordinators in MAINTAINERS.txt:
Core mentoring coordinators
---------------------------
The Drupal Core mentors inspire, enable, and encourage new core contributors.
See https://www.drupal.org/core-mentoring for more information about mentoring.
Mentoring coordinators recruit and coach other mentors. They work on contributor
tools, documentation, and processes to make it easier for new contributors to
get involved. They organize communications and logistics, and actively
participate in mentoring.
- Mauricio Dinarte 'dinarcon' https://www.drupal.org/u/dinarcon
- Lucas Hedding 'heddn' https://www.drupal.org/u/heddn
- Tara King 'sparklingrobots' https://www.drupal.org/u/sparklingrobots
- Rachel Lawson 'rachel_norfolk' https://www.drupal.org/u/rachel_norfolk
- Elli Ludwigson 'ekl1773' https://www.drupal.org/u/ekl1773
- Jess Myrbo 'xjm' https://www.drupal.org/u/xjm
- Matthew Radcliffe 'mradcliffe' https://www.drupal.org/u/mradcliffe
- (provisional) Chris Darke 'chrisdarke' https://www.drupal.org/u/chrisdarke
- (provisional) AmyJune Hineline 'volkswagenchick' https://www.drupal.org/u/volkswagenchick
Provisional membership: None at this time.
Merge request to follow removing Provisional membership: None at this time.
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | 3195951-9.patch | 415 bytes | Pooja Ganjage |
Issue fork drupal-3195951
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
changes
Comments
Comment #3
volkswagenchickThanks, I am adding the related issue.
Comment #5
thejimbirch CreditAttribution: thejimbirch at Kanopi Studios commentedMerge request in. Diff looks like I expected. Setting as Needs Review:
Comment #6
thejimbirch CreditAttribution: thejimbirch at Kanopi Studios commentedComment #7
volkswagenchickMoving to RTBC . Thanks Jim
Comment #8
alexpottNeeds 9.2.x to be merged and conflicts fixed... I can't apply the plain diff... https://git.drupalcode.org/project/drupal/-/merge_requests/286.diff
Comment #9
Pooja Ganjage CreditAttribution: Pooja Ganjage at Asentech LLC commentedHi,
Creating a patch for this issue.
Please review the patch once.
Thanks.
Comment #10
Pooja Ganjage CreditAttribution: Pooja Ganjage at Asentech LLC commentedComment #11
volkswagenchickComment #12
thejimbirch CreditAttribution: thejimbirch at Kanopi Studios commentedI rebased 9.2.x in which includes 165a9089 that was merged last night.
https://git.drupalcode.org/project/drupal/-/merge_requests/286.diff
Comment #13
Gábor HojtsyPer discussion in #3191699: MAINTAINERS.txt role descriptions outdated, topic maintainer description missing, provisional maintainership incorrect all instances of "Provisional membership: None at this time." in the file should be removed for consistency. I personally have nothing against removing only for this section, but @xjm did not agree with this there. Since that issue had a larger scope and we did not agree on scope, keeping this but moving to needs work for removing all "Provisional membership: None at this time." in this issue.
Comment #15
Gábor HojtsyAlso crediting @ChrisDarke from #3187927: Make provisional maintainership appear consistently in MAINTAINERS.txt which is essentially the same issue and predates this with two months. It is my mistake that I tried to bundle up what looked like minor text changes to me in #3191699: MAINTAINERS.txt role descriptions outdated, topic maintainer description missing, provisional maintainership incorrect, resulting now in the third issue to deal with the same thing. Sorry.
Comment #16
thejimbirch CreditAttribution: thejimbirch at Kanopi Studios commentedAdded 1 commit, 9d3e99a6 Removes all "Provisional membership: None at this time." per Gábor Hojtsy in #13. That includes Usability and Media Initiative, in addition to Core mentoring coordinators which the original commit removed. I also rebased additional commits in to bring 9.2.x up to date.
Here's the diff:
https://git.drupalcode.org/project/drupal/-/merge_requests/286.diff
Comment #17
xjmI'd like to propose an alternate solution with a somewhat wider scope. The "Provisional membership: none at this time." lines were added on a whim in 2015 when we updated the governance, with the notion that mentioning it might raise awareness that provisional maintainership was something people could do. However, since then we've seen that these sections are consistently not kept up to date. The formatting is also confusing for the topic and initiative maintainers' respective versions of this line.
Since it's proven difficult to keep accurate, and the value it adds is limited, I propose we remove the line everywhere it is used in the file.
Comment #18
Gábor HojtsyExactly that is done in the MR :) So this looks good then.
Comment #19
alexpottCommitted 4c176c3 and pushed to 9.2.x. Thanks!
Committed 7881deb and pushed to 9.1.x. Thanks!