Skip to content
This repository has been archived by the owner on Aug 31, 2021. It is now read-only.

Add Code of Conduct #250

Closed
wants to merge 5 commits into from
Closed

Add Code of Conduct #250

wants to merge 5 commits into from

Conversation

flarnie
Copy link
Contributor

@flarnie flarnie commented Dec 20, 2017

In the past Facebook didn't promote including a Code of Conduct when creating new projects, and many projects skipped this important document. Let's fix it. :)

why make this change?:
Facebook Open Source provides a Code of Conduct statement for all
projects to follow, to promote a welcoming and safe open source community.

Exposing the COC via a separate markdown file is a standard being
promoted by Github via the Community Profile in order to meet their Open
Source Guide's recommended community standards.

As you can see, adding this file will improve the Surround360 community profile
checklist
and increase the visibility of our COC.

test plan:
Viewing it on my branch -
screen shot 2017-12-20 at 7 57 00 am
screen shot 2017-12-20 at 8 38 58 am

issue:
internal task t23481323

In the past Facebook didn't promote including a Code of Conduct when creating new projects, and many projects skipped this important document. Let's fix it. :)

why make this change?:
Facebook Open Source provides a Code of Conduct statement for all
projects to follow, to promote a welcoming and safe open source community.

Exposing the COC via a separate markdown file is a standard being
promoted by Github via the Community Profile in order to meet their Open
Source Guide's recommended community standards.

As you can see, adding this file will improve the [Surround360 community profile
checklist](https://github.com/facebook/Surround360/community) and increase the visibility of our COC.

test plan:
Viewing it on my branch -
(Flarnie will insert screenshots)

issue:
internal task t23481323
We just added a `CODE_OF_CONDUCT.md` and it makes sense to surface it here.
We just added a `CODE_OF_CONDUCT.md` and it makes sense to surface it here.
@aparrapo
Copy link
Contributor

Thanks for adding it @flarnie! :)

Copy link

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@aparrapo is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@flarnie
Copy link
Contributor Author

flarnie commented Dec 20, 2017

Thank you for the quick review @aparrapo :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants