Skip to content
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

Update values #2490

Merged
merged 1 commit into from
Feb 5, 2021
Merged

Update values #2490

merged 1 commit into from
Feb 5, 2021

Conversation

mercadon
Copy link
Contributor

@mercadon mercadon commented Feb 3, 2021

Replace "be a good human" with "be welcoming and inclusive"

Replace "be a good human" with "be welcome and inclusive"
@sourcegraph-bot
Copy link
Contributor

Notifying subscribers in CODENOTIFY files for diff c817e22...2794e0d.

Notify File(s)
@christinaforney company/values.md
@jeanduplessis company/values.md
@nicksnyder company/values.md
@sqs company/values.md

@virginiaulrich
Copy link
Contributor

Approved, though do wonder if it should be "be welcoming and inclusive ..." Not sure which is easier to read, so that doesn't change my review, but will comment in case anyone else has a stronger sense of which reads better

@dadlerj
Copy link
Member

dadlerj commented Feb 3, 2021

Thanks for the thoughtful wording and structure here!

Copy link
Contributor

@nicksnyder nicksnyder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Love it, and I think we can/should link directly to this from our job descriptions.

@mercadon
Copy link
Contributor Author

mercadon commented Feb 3, 2021

All credit goes to @virginiaulrich on the thoughtful update and rewording of this value! 🥇

Copy link
Contributor

@jeanduplessis jeanduplessis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Really happy to see this update ❤️

@mercadon mercadon merged commit 6a77a42 into main Feb 5, 2021
@mercadon mercadon deleted the mercadon-patch-13 branch February 5, 2021 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants