-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Vitess Slack Guidelines v1.0 #6961
Conversation
Signed-off-by: Alkin Tezuysal <alkin.tezuysal@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we add attribution? That this is adapted from https://github.com/cncf/foundation/blob/master/slack-guidelines.md?
doc/VitessSlackGuidelinesv1_0.md
Outdated
### Unwanted Voice/Video Calling | ||
Abuse of the voice/video calling features to call users without their permission is not allowed. Accidents sometimes happen, let the user know it was a mistake. | ||
|
||
## Reporting a Problem, Issue ,or Bug |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
## Reporting a Problem, Issue ,or Bug | |
## Reporting a Problem, Issue, or Bug |
doc/VitessSlackGuidelinesv1_0.md
Outdated
## Channel Spamming | ||
Slack is our best communication medium for the community. Please keep your messages on topic for each channel, and please use the appropriate channel. Spamming channels with unrelated or marketing messages is not allowed. | ||
|
||
## Unwanted Solitication through Private Messages |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo: Solitication => Solicitation
This feels more like website content and maybe not so much something that belongs here, but I'm not sure. What was the thinking for adding it here, where it probably won't be seen much? |
Once it is merged we'll publish it on vitess.slack.com and link to it in the #general channel. |
Approved, but like @systay I feel like this belongs in a different repo than |
Signed-off-by: Alkin Tezuysal <alkin.tezuysal@gmail.com>
All Vitess documentation except the Website is in vitessio/vitess repo as far as I know. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Fair. This makes sense. |
@deepthi should we ask a non-PS maintainer to review this? |
Not necessary, because the content originates from CNCF.
There's probably a better place to put this, but this is a good start. We can move it later as we reorganize things. |
Signed-off-by: Alkin Tezuysal alkin.tezuysal@gmail.com