Skip to content

Conversation

@story645
Copy link
Member

Added the mailing list for the social media contributors

Added the mailing list for the social media contributors
timhoffm
timhoffm previously approved these changes Aug 9, 2022
dopplershift
dopplershift previously approved these changes Aug 15, 2022
@story645 story645 dismissed stale reviews from dopplershift and timhoffm via 7b44f55 September 8, 2022 03:48
@story645 story645 changed the title Added social media mailing list Added social media mailing list & gitter channel Sep 8, 2022
@story645
Copy link
Member Author

story645 commented Sep 8, 2022

thansk for your approvals @timhoffm and @dopplershift, I updated this to include the gitter channel too since that seemed more sensible than opening a new PR that I'd have to rebase on this one anyway

@timhoffm
Copy link
Member

timhoffm commented Sep 8, 2022

I'm unclear what the merge policies here are. I'd say this is doc-like,so one approval is enough. OTOH these are the governance docs which maybe need a majority of the steering council?

@story645
Copy link
Member Author

story645 commented Sep 8, 2022

Yeah I put this doc in this repo before the 3 merge rule and #24 is to move it out but the repo to put it in doesn't exist yet. I agree that this should be mergeble with one approval.

Frankly I'm uncomfortable w/ the fact that technically the communications lead (community manager since it's currently under that umbrella) doesn't even have to be looped into changes on this doc, given they'd be the one charged w/ implementing any new policies. And yes I wrote this doc so that's on me, but now I'm unsure how a PR making that clear would be received.

@dopplershift
Copy link
Contributor

@story645 I'd be receptive to a PR clarifying it.

@dopplershift dopplershift merged commit a252b01 into matplotlib:main Sep 22, 2022
@story645 story645 deleted the comms-mailing-list branch September 22, 2022 23:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants