-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
WEB: Add greeting note to CoC #32024
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
Conversation
Should be good - can you merge master to get green? |
aa3a7a0
to
f5d3922
Compare
web/pandas/about/team.md
Outdated
@@ -42,6 +42,7 @@ If you want to support pandas development, you can find information in the [dona | |||
> or anyone willing to increase the diversity of our team. | |||
> We have identified visible gaps and obstacles in sustaining diversity and inclusion in the open-source communities and we are proactive in increasing | |||
> the diversity of our team. | |||
> We welcome inclusive behavior - for example, please don’t say “hey guys!” but “hey everyone!”. |
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.
This is the team page, doesn't seem relevant to me here. Should it be in the code of conduct instead?
I agree this feels a bit "out of place" where it is added now |
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, thanks @MarcoGorelli
If you can merge master, looks like something (I guess unrelated) failed. |
873ae61
to
c0aea1e
Compare
Thanks @MarcoGorelli |
I noticed something similar in the CPython contributing docs:
and thought it was really nice.
Pushing directly rather than opening as good first issue to avoid invoking a s*******m