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

Simplify and concrete #7

Merged
merged 1 commit into from
Mar 20, 2018
Merged

Simplify and concrete #7

merged 1 commit into from
Mar 20, 2018

Conversation

linkita
Copy link
Contributor

@linkita linkita commented Dec 17, 2017

Hello folks! I did a new version of the CoC. I changed a few things that basically simplify, clarify and structure them.

I removed subjective sentences like "reasonably be considered..." and replaced them with objetive words like violence. The same case with the Enforcement: I would like to be clear: We will help you. I don´t like impersonal sentences like "will investigate... will review...". because this can be putting in doubt our implication with this CoC.

@ignasi35
Copy link
Contributor

Hi @linkita this looks great.

Recently a non-personal account representing a company showed up trying to hire people. While 'm not against users promoting their job offers on the appropriate channels I think using non-personal accounts is a no-no. I'm not sure how to reflect that on this CoC. WDYT?

@gonzaloserrano gonzaloserrano merged commit 56b6e96 into bcneng:add/coc Mar 20, 2018
@gonzaloserrano gonzaloserrano mentioned this pull request Mar 20, 2018
@gonzaloserrano
Copy link
Contributor

I've created #8 to reflect the issue commented by Ignasi.

gonzaloserrano added a commit that referenced this pull request Mar 20, 2018
* Add CoC v1

* Adds `technology` to the list

* Revert "Adds `technology` to the list" (#3)

* Simplify and concrete (#7)
@linkita linkita deleted the add/coc branch November 13, 2019 22:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants