Skip to content

Conversation

shrugs
Copy link
Contributor

@shrugs shrugs commented Jul 4, 2018

Description

Adds the contributor covenant to our open source project!

@frangio
Copy link
Contributor

frangio commented Jul 4, 2018

Not sure why but this one is slightly different than the one found at the site. It's odd because it's the same version.

Can we use that one verbatim?

@shrugs
Copy link
Contributor Author

shrugs commented Jul 4, 2018

@frangio which parts stand out to you? I don't see it after a quick eyeball

@nventuro
Copy link
Contributor

nventuro commented Jul 4, 2018

@shrugs yours is missing this section:

Project maintainers who do not follow or enforce the Code of Conduct in good faith may face temporary or permanent repercussions as determined by other members of the project's leadership.

and says

regardless of age, body size, disability, ethnicity, gender identity and expression, level of experience, nationality, personal appearance, race, religion, or sexual identity and orientation.

instead of

regardless of age, body size, disability, ethnicity, sex characteristics, gender identity and expression, level of experience, education, socio-economic status, nationality, personal appearance, race, religion, or sexual identity and orientation.

@shrugs
Copy link
Contributor Author

shrugs commented Jul 4, 2018

gotcha! I'll manually copy and paste the md file from the website instead of using github's interface

Copy link
Contributor

@frangio frangio left a comment

Choose a reason for hiding this comment

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

Thanks @shrugs!

@shrugs
Copy link
Contributor Author

shrugs commented Jul 4, 2018

is the merge button broken for y'all as well? all green but "required status must pass before merging"

@frangio
Copy link
Contributor

frangio commented Jul 5, 2018

Broken for me too 😢.

I'll force merge it.

@frangio frangio merged commit 4575a24 into master Jul 5, 2018
@frangio frangio deleted the feat/add-code-of-conduct-1 branch July 5, 2018 18:44
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.

3 participants