Skip to content

Removing Sam as Doc WG lead#287

Merged
knative-prow-robot merged 2 commits intoknative:masterfrom
RichieEscarez:RichieEscarez-patch-1
Sep 23, 2020
Merged

Removing Sam as Doc WG lead#287
knative-prow-robot merged 2 commits intoknative:masterfrom
RichieEscarez:RichieEscarez-patch-1

Conversation

@RichieEscarez
Copy link
Copy Markdown
Contributor

Sam stopped working on the project some time in March.

@googlebot googlebot added the cla: yes Indicates the PR's author has signed the CLA. label Sep 22, 2020
@knative-prow-robot knative-prow-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Sep 22, 2020
@RichieEscarez
Copy link
Copy Markdown
Contributor Author

assign/ @abrennan89

@mattmoor
Copy link
Copy Markdown
Member

Hey @RichieEscarez, welcome back. Sorry to hear that @samodell might not come back, I hope she's doing better.

The process for new working group leads is spelled out here, and the PRs adding new leads are generally asked to follow a form like this. I think the idea of "replacing" WG leads sets a bad precedent, so I'd prefer if we removed Sam separately from the process above.

I don't think the quantitative aspects of our WG lead "template" cover everything though, especially prolonged absence. I think I'd personally be more comfortable if the bulk of the contributions justifying WG leadership had a bias towards recent contribution because in that role the importance of context is fairly high.

@RichieEscarez
Copy link
Copy Markdown
Contributor Author

Thanks Matt! Sam is definitely off the project. She is back to being "Sam" but decided to move back into working on GCP products rather than OSS.

Sounds good (to just removing Sam in this PR). My motivation here had two parts. I added this to todays Docs WG meeting with the intention to discuss this there but given the lack of sessions and attendance (today and through Aug), I knew that I would/could get input/feedback this route.

Question: Given the two lead definitions, my goal was to try and help fulfill the expectations listed. I know much as changed since the beginning of the year especially in the Docs WG and many formal WG processes seem not to exist or are lacking an owner. That type of support is not articulated in the community guidelines so I'm wondering if it would be considered?

Also, regarding "absence", that's not spelled out either? I did ramp down significantly but was still an active member over Q2 and Q3. That said, the scope of time for someones contributions are also lacking in the community guidelines (does that mean I still can check all the boxes)? Anyway, ill try again and bring up whether I should fill out the Docs WG lead from in the next Docs meeting.

@RichieEscarez RichieEscarez changed the title replacing Sam as Doc WG lead Removing Sam as Doc WG lead Sep 22, 2020
@evankanderson
Copy link
Copy Markdown
Member

Welcome back @RichieEscarez !

I agree with Matt that making this two PRs will be clearer, and I'm sorry to hear that we won't be seeing Sam again.

Given your previous time with the project, I'd expect that you'd probably be quickly contributing effectively as a WG lead -- I'd defer decision on the nomination to @abrennan89 , who has been trying to right the process ship that slipped during the time when there was no WG lead.

I'd advise her to sit down with you and make sure that you two have the same vision and goals (possibly by working together for at least a few weeks) before adding a second WG lead. Having an extra set of hands to bail out a boat can be helpful, unless you each end up bailing into each others' boats (i.e. working at cross-purposes). Working together for a few weeks and planning out what the next few months look like should help both of you figure out if it's a good fit, since I know Ashleigh has a few opinions about what her ideal stack would look like.

Copy link
Copy Markdown
Member

@evankanderson evankanderson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
/approve

Since this is a pure removal.

I can follow up with a PR for peribolos.

@knative-prow-robot knative-prow-robot added the lgtm Indicates that a PR is ready to be merged. label Sep 23, 2020
@knative-prow-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: evankanderson, RichieEscarez

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@knative-prow-robot knative-prow-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 23, 2020
evankanderson pushed a commit to evankanderson/community that referenced this pull request Sep 23, 2020
@knative-prow-robot knative-prow-robot merged commit 8900aef into knative:master Sep 23, 2020
@RichieEscarez RichieEscarez deleted the RichieEscarez-patch-1 branch March 8, 2021 22:22
daisy-ycguo pushed a commit to daisy-ycguo/community that referenced this pull request Mar 12, 2021
* Add self to the TOC (it's that easy!).

* And fix Louis' avatar
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cla: yes Indicates the PR's author has signed the CLA. lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants