Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Update comms guidelines #36

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 5 commits into from
Closed

Conversation

story645
Copy link
Member

Bunch of little changes cause this doc was getting way out of date.

  • Added/updated all the accounts I know of just so they're in one place - I think @mdboom has access to youtube?

  • notice of how to get access to accounts

  • updated coc link - it had been old one

Um I kinda had this open for 3 weeks since the call where it was discussed, but on my fork of the governance repo 🤦‍♀️

Bunch of little changes cause this doc was getting way out of date. 

* Added/updated all the accounts I know of just so they're in one place - I think @mdboom has access to youtube?

* notice of how to get access to accounts

* updated coc link - it had been old one
dopplershift
dopplershift previously approved these changes Jun 22, 2023
timhoffm
timhoffm previously approved these changes Sep 4, 2023
@story645
Copy link
Member Author

story645 commented Sep 4, 2023

I have no idea why this doc build is failing - I've tried three different linkings that have worked just fine in preview mode.

@QuLogic
Copy link
Member

QuLogic commented Sep 5, 2023

Isn't it this,

WARNING: Path to dark image logo does not exist: images/logo_dark.svg

aka #34?

@story645
Copy link
Member Author

story645 commented Sep 5, 2023

Isn't it this,

WARNING: Path to dark image logo does not exist: images/logo_dark.svg

aka #34?

Probably, which is partially why I don't think the must have approval of min_3(steering_count) rule should be hardcoded. Don't really think this one should need that either since it's documenting current practice :/

@timhoffm
Copy link
Member

timhoffm commented Oct 5, 2023

Can this be closed in favor of #26703?

@story645
Copy link
Member Author

story645 commented Oct 5, 2023

Yeah, was leaving this open til there was consensus on #26703 going in but #26703 supercedes the comms PRs here. (I can make both of the ones here draft for now)

@story645 story645 marked this pull request as draft October 5, 2023 14:22
@story645
Copy link
Member Author

story645 commented Nov 1, 2023

superceded by matplotlib/matplotlib#26703

@story645 story645 closed this Nov 1, 2023
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