Add Brian Gilbert (realityloop) as a provisional mentoring coordinator

I was surprised Brian wasn't already listed! He's been making our mentoring tools and training awesome since forever.

The next step is for @realityloop to confirm on the issue that he accepts the role as described in MAINTAINERS.txt:

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.

As well as the general maintainer role as described in the core governance.

And we'll need a patch against MAINTAINERS.txt. :)

Hooray!

Hi,

Creating a patch to include Brian Gilbert (realityloop) as a provisional mentoring coordinator in MAINTAINERS.txt.

Please review the patch.

Thanks.

Thanks! Still waiting on Brian to actually give written confirmation first on this.

Adding a related issue. Wondering if all three issues should be addressed in one patch?

Moving this to needs work as Brian has yet to comment.

I did think about that but since they were separate issues I wasn't sure what the protocol was to bundle them together.

Hi,

Creating a patch as added all three in one as a provisional mentoring coordinator in MAINTAINERS.txt.'

Please review the patch.

Thanks.

Hi @pooja-ganjage we are still waiting on @realityloop to confirm this nomination, so as such it's a bit premature to push up a patch for this. Doubt he will NOT confirm, but just to keep the order of events right.
I am open to debate on this though of course!

Thanks for the patch. I do think for consistency the names should be in alphabetical order by last name as the list above this section. thanks.

Example.
- 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

Created patch as suggested in #12 comment.

Thanks.

@Pooja Ganjage, thanks for your help, but let's stop posting patches until we have feedback from Brian on the issue. The appointees are independent of each other on purpose because it's not a "package deal" -- only one might accept the role, or all three, etc. :) So let's wait for now rather than rolling patches that might just need to be rerolled anyway.

@rachel_norfolk @xjm happy to accept addition, apologies for the delay in response.

Needs Review as @realityloop accepted as a provisional mentoring coordinator.

@Pooja Ganjage, regarding https://www.drupal.org/project/drupal/issues/3187241#comment-13933959, the commits are being kept seperately, and per @xjm, adding them into the main listing with (provisional) in front

Congratulations @realityloop !

@realityloop I think we need a comment like #3187239-6: Add Chris Darke (ChrisDarke) as a provisional mentoring coordinator to agree to everything outlined by @xjm in #3

Hi Pooja, what Alex was referring to in their comment was that Brian needs to add in a more detailed acceptance of the task he is being asked to perform. Something along the lines of what I wrote in mine:

"I confirm that I accept the role as described in MAINTAINERS.txt, and the general role of a maintainer as covered in the core governance."

Until he does that we cannot move forwards with any updates to this issue. I thought that his previous comment in #15 was enough but it was not. Let me know if you have any questions about this :)

sorry for the delay, we need ability to set notifications on @mentions on D.o!

I confirm that I accept the role as described in MAINTAINERS.txt, and the general role of a maintainer as covered in the core governance.

Thanks Brian!!

I am moving this issue to Needs work. The patch/merge request will have to updated due to recent updates to that file.

Thanks!

Seeing 135 commits was a bit weird, but yes, ultimately, the only code change is what is needed for this issue. My guess is the merging of changes to the main branch is not relevant in the end, and only on the fork.

I am going to change this to RTBC to see if a maintainer agrees.

Added Related issue per the comment in #27

Committed and pushed 7ccaa8faa5 to 9.2.x and db5f7f243b to 9.1.x. Thanks!

@rachel_norfolk I had created an issue regarding tidying up Maintainers.txt and that lead into a longer discussion about what the entries should say, and this was merged by Gabor into this issue https://www.drupal.org/project/drupal/issues/3191699