New Maintainer Checklist: add "real" name, meetup suggestion. #6185
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
issyl0
reviewed
May 29, 2019
vitorgalvao
reviewed
May 29, 2019
vitorgalvao
reviewed
May 29, 2019
docs/New-Maintainer-Checklist.md
Outdated
| @@ -79,6 +79,8 @@ If they are elected to the Homebrew's [Project Leadership Committee](https://doc | |||
|
|
|||
| If there are problems, ask them to step down as a maintainer and revoke their access to all of the above. | |||
|
|
|||
| In interests of loosely verifying maintainer identity and building camaraderie, if you find yourself in the same town (e.g living, visiting or at a conference) as another Homebrew maintainer you should make the effort to meet up. If you do so, you can expense your meal (within SFC policies). This is a more relaxed version of e.g. the Debian system to meet in person to sign keys with a legal ID verification. | |||
There was a problem hiding this comment.
To clarify, we’re talking about expensing the meals for both parties? This is quite a fun idea!
There was a problem hiding this comment.
Yup! That's what we've done at GitHub also as a gentle nudge and, with a peek at Homebrew's finances, we have enough funds to do that if people aren't being silly about it.
There was a problem hiding this comment.
@alebcay Durham Homebrew meetup? 😀
There was a problem hiding this comment.
@zachauten Having neglected my GitHub notifications for about five months 😅, I've missed this, but if you're still up for it, let's meet up sometime!
vitorgalvao
approved these changes
May 29, 2019
GauthamGoli
approved these changes
May 29, 2019
SMillerDev
approved these changes
May 29, 2019
zachauten
approved these changes
May 29, 2019
sjackman
approved these changes
May 29, 2019
scpeters
approved these changes
May 29, 2019
Moisan
approved these changes
May 29, 2019
jonchang
approved these changes
May 29, 2019
claui
approved these changes
May 29, 2019
colindean
approved these changes
May 29, 2019
iMichka
approved these changes
May 29, 2019
We've had problems in the past with Homebrew maintainers being unwilling to disclose their identity or meet other maintainers so here are some very light-touch suggestions to address this in future. These ideas were run past the PLC and agreed there but I still welcome input from maintainers.
Co-Authored-By: Colin Dean <[email protected]>
MikeMcQuaid
force-pushed
the
new-maintainer-checklist-tweaks
branch
from
00f6360 to
42c8861
Compare
|
Thanks for review and comments all! |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
We've had problems in the past with Homebrew maintainers being unwilling to disclose their identity or meet other maintainers so here are some very light-touch suggestions to address this in future.
These ideas were run past the PLC and agreed there but I still welcome input from @Homebrew/maintainers.
brew stylewith your changes locally?brew testswith your changes locally?