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

RFC: Updates to Community Code of Conduct #1467

Merged
merged 1 commit into from
May 31, 2017
Merged

Conversation

eeyun
Copy link
Contributor

@eeyun eeyun commented Nov 9, 2016

This is the first pass on a PR that more explicitly defines the boundaries of our community interactions.

What we had was not bad by any stretch of the imagination. But I find myself wanting something more explicit in both the process and expectation around how our community handles itself.

These changes should not realistically be anything new to the community, however a PR of this nature should more clearly delineate what is accepted/expected from community members and that's what I'm hoping to do here.

PLEASE LEAVE COMMENTS AND ADD REVIEWS.

I can't personally think of a PR that is more important to Habitat than a PR defining what we want from our community.

@thesentinels
Copy link
Contributor

Thanks for the pull request! Here is what will happen next:

  1. Your PR will be reviewed by the maintainers
  2. If everything looks good, one of them will approve it, and your PR will be merged.

Thank you for contributing!


In the interest of fostering an open and welcoming environment, we as
In the interest of fostering an open, welcoming, and encouraging environment, we as
Copy link

Choose a reason for hiding this comment

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

welcoming, encouraging, and supportive environment?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i think encouraging presupposes supportive?

Copy link

Choose a reason for hiding this comment

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

For me, just "open and welcoming" is the most clear variant.

Copy link

Choose a reason for hiding this comment

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

Open and welcome does not include encouraging. Encouraging sets a standard for mental/emotional support in terms of language choices when interacting with folks who bring new or less than perfect contributions to the table.

Copy link
Contributor

Choose a reason for hiding this comment

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

I'm fond of encouraging as well - and for the record, Habitat has been one of the most encouraging, welcoming and open communities I've ever participated in.

* Your work will be used by other people, and you, in turn, will depend on the work of others.
* Decisions that you make will often affect others in the community.
* Disagreements happen, but should not be an excuse for poor behavior and bad manners. When disagreements do happen, let’s work together to solve them effectively and in a way that ensures that everyone understands what the disagreement was.
* People may not understand jokes, sarcasm, and oblique references in the same way that you do. So remember that and be kind to the other members of the community.
Copy link

Choose a reason for hiding this comment

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

Out of principle i get this, but it's hard to gauge what's considered ok here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure, I don't think this section is about hard values though. I think this is more of a: things to keep in mind while you're interacting with other community members. Perhaps we should make that more clear?

Choose a reason for hiding this comment

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

make amends can be tricky and can often make things worse - we should be careful here, this is where I'd want Code of Conduct crew or someone trained to deal with these things sitting

Copy link

Choose a reason for hiding this comment

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

I think make amends is right. Maybe borrow from Kronda's tweet here https://twitter.com/kronda/status/546727506850496512

Copy link
Contributor

@miketheman miketheman Nov 11, 2016

Choose a reason for hiding this comment

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

I'll agree with the sentiment of the tweet, need to rephrase it.

How about a bit of paraphrasing:

  • Our community spans languages, cultures, perspectives, and many other facets, and sometimes inside jokes, sarcastic comments, or oblique references may not be perceived as planned by the author. Keep this in mind, and be kind to others, treat them with the same respect you would expect to receive.

* Consult with another maintainer to make a judgement call about what reasonable corrective actions are warranted.
* Apply the corrective action.
* Document the incident.

Copy link

Choose a reason for hiding this comment

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

Should we put in a standard email or google voice number to call and call it out here? If we standardize on it, it could in the case of an incident to say "email this" "call that number" and you'll find someone to help.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ideally what we are moving towards is something similar to the way Chef handles this. We will likely have contributors and maintainers that exist as community advocates who can field these things. Once we've got folks who are willing to do that, we'll create a chart in here that explains who those people are. For now though maybe it'd be good for us to link to the maintainers doc?

Copy link

Choose a reason for hiding this comment

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

If it's not explicit how to get help, then as a whole not helpful. Doesn't show follow-through. There should also be a companion to this policy with training for folks on how to handle different scenarios.

@cheeseplus
Copy link

Added some comments but otherwise +1

* Using welcoming and inclusive language
* Exercising patience and friendliness
* Being respectful of differing viewpoints and experiences
* Gracefully accepting constructive criticism
* Focusing on what is best for the community
* Showing empathy towards other community members

Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe try active voice here? For example, "Use welcoming and inclusive language" and "Exercise patience and friendliness".

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah I wondered about that, I think that would be a good change thank you!

Examples of behavior that contributes to creating a positive environment
include:
Be careful in the words that you choose. Be kind to others. Practice empathy. Don't
insult or put down others. Remember that sexist, racist, and other exclusionary jokes
Copy link

Choose a reason for hiding this comment

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

sexist, racist, ableist

* Decisions that you make will often affect others in the community.
* Disagreements happen, but should not be an excuse for poor behavior and bad manners. When disagreements do happen, let’s work together to solve them effectively and in a way that ensures that everyone understands what the disagreement was.
* People may not understand jokes, sarcasm, and oblique references in the same way that you do. So remember that and be kind to the other members of the community.
* Sexist, racist, and other prejudicial or exclusionary comments are not welcome in the community.
Copy link

Choose a reason for hiding this comment

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

Sexist, racist, ableist


#### Documenting Incidents

All incident reports will be kept in a private repository that is shared with the advocates, maintainers and Community Manager.
Copy link

@iennae iennae Nov 11, 2016

Choose a reason for hiding this comment

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

This seems scary. This needs to be an explicit list and minimal if it has identifying information in it. It needs to be stuck in a moment of time where someone new doesn't get access to it.

Copy link
Contributor

Choose a reason for hiding this comment

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

Can you elaborate on this a bit? It seems like having historical context of incidents is useful in determining further actions.

Copy link

Choose a reason for hiding this comment

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

Sure. If I report an incident, I want to know exactly who is going to know about and what they are going to do with that information. If this is something that is going to have identifying information over a long haul, this means that I might be more hesitant to report something if I don't have some kind of knowledge about who might have access to that information. For example, how do I know that in 6 months someone who was involved with the incident doesn't get access to the details?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Totally that makes sense. I was also talking with @Hoverbear about this. I think this section should change to include information on how to report an incident (and who SPECIFICALLY to report it to) within the community and information for how to report in the case that the person in question (the individual who is breaking the community covenant) are one in the same. E.g. John Doe is a community advocate I should report harassment to, but John Doe is the person doing the harassing in this case. And that second point of contact should be outside of the team that typically handles these things.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

So essentially what I'm saying is @iennae is right. The list of people that have access to the incident lists needs to be more explicit and significantly less inclusive.

Choose a reason for hiding this comment

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

I fully agree with @iennae :)

@bixu
Copy link
Contributor

bixu commented Nov 11, 2016

I think I agree here - fostering learning environments is important (and also helps surface less obvious pain points in what we're building).

On 11 Nov 2016, at 18:55, Jennifer Davis [email protected] wrote:

@iennae commented on this pull request.

In CODE_OF_CONDUCT.md:

-In the interest of fostering an open and welcoming environment, we as
+In the interest of fostering an open, welcoming, and encouraging environment, we as
Open and welcome does not include encouraging. Encouraging sets a standard for mental/emotional support in terms of language choices when interacting with folks who bring new or less than perfect contributions to the table.


You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


As you are working with other members of the community, please keep in mind the following guidelines, which apply equally to founders, mentors, those who submit new features and pull requests, and to anyone who is seeking help and guidance.
Copy link
Contributor

Choose a reason for hiding this comment

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

Minor style nitpick: other lines terminate around the 80 char mark, these do not.

* Trolling, insulting/derogatory comments, and personal or political attacks
* Public or private harassment
advances.
* Violent threats or language directed against another person.
Copy link
Contributor

Choose a reason for hiding this comment

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

What about non-violent threats?

  • Threats or violent language directed against another person

advances.
* Violent threats or language directed against another person.
* Posting sexually explicit or violent material.
* Sexist, racist, homophobic, transphobic, ableist or otherwise discriminatory jokes and language.

Choose a reason for hiding this comment

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

"...transphobic, ableist, or otherwise..." don't leave off the oxford comma.

@eeyun
Copy link
Contributor Author

eeyun commented Nov 24, 2016

Iteration 2 of this is inbound early next week. We will be correcting some of the verbage as discussed above as well as defining a community hierarchy for reporting and handling incidents. I'll post here when the new commit is in :)

@eeyun
Copy link
Contributor Author

eeyun commented May 11, 2017

OK. SO. Theres a new Code of Conduct iteration that I've just pushed to this branch. I would super appreciate if those of you who previously had comments would check the new CoC to determine if your primary concerns have been resolved.

Important changes to note: We now have an explicit list of Community Organizers! Huzzah! We also now have an explicit guide for how to handle an incident with help from those community organizers! Huzzah! I've tightened verbage based on the previous concerns listed. Assuming there are no greater concerns this is the content that I think we should use.

* Your work will be used by other people, and you, in turn, will depend on the work of others.
* Decisions that you make will often affect others in the community.
* Disagreements happen, but should not be an excuse for poor behavior and bad manners. When disagreements do happen, let’s work together to solve them effectively and in a way that ensures that everyone understands what the disagreement was.
<<<<<<< HEAD
Copy link
Contributor

Choose a reason for hiding this comment

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

Did you mean to leave this in? :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Whoops!

representing a project or community include using an official project e-mail
address, posting via an official social media account, or acting as an appointed
representative at an online or offline event. Representation of a project may be
further defined and clarified by project maintainers.
further defined and clarified by project maintainers and community organizers.

## Enforcement
Copy link

Choose a reason for hiding this comment

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

The mechanism to report feels like it should be nearer to the top.

* The behavior that was in violation
* The approximate time of the behavior
* The circumstances surrounding the incident
* Other people involved in the incident
Copy link

Choose a reason for hiding this comment

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

Might want to be clear about whether the victim's information will be stored in repository.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

My initial feeling is to say that we should track an initial point of contact for the victim. Do you have any opinions on that?

Copy link

Choose a reason for hiding this comment

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

I think the victims information shouldn't be included in the repo and we should explicitly say that. Tracking the information outside of the event of that detail is important, but leaving a paper trail about the victim decreases the safety around reporting events to some degree.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Perhaps it does. I've been feeling mixed about it. If you and others feel strongly this is the right way to go then I'm in on it.


* Consult with another Community Organizer to make a judgement call about what reasonable corrective actions are warranted.
* Apply the corrective action.
* Document the incident.
Copy link

Choose a reason for hiding this comment

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

Document the incident as described below.

If you have any lack of clarity about behaviors we include in the definition of "harassment", please read the [Citizen Code of Conduct](http://citizencodeofconduct.org/). In particular, we don’t tolerate behavior that excludes people in socially marginalized groups.

### Roles
The following are the various roles of our Community Organizers and the people assigned to each role:
Copy link

Choose a reason for hiding this comment

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

maybe bold Community Organizers here at the top or have a separate paragraph to define them as the collection of all roles. Later when you use Community Organizers it's a little confusing as I was looking for the specific roles here.


The Advocate should take the following steps if the behavior is not brought inline with our guidelines or the incident is not resolved.

* Consult with another Community Organizer to make a judgement call about what reasonable corrective actions are warranted.
Copy link

Choose a reason for hiding this comment

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

This might need to be expanded out a little. What if 1 organizer thinks that the situation doesn't warrant action? "Consult with at least one other Community Organizer to identify the set of correction actions to be taken.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes excellent. The process should be an escalation of the snowball variety I think. E.g. if two cant agree, get a third, if 3 cant agree get a 4th, if 4 cant agree get the Decider.

### Roles
The following are the various roles of our Community Organizers and the people assigned to each role:
* The <b>Decider</b> has final say on community guidelines and final authority on correct actions and appeals. The top-level decider is [Adam Jacob]([email protected]), [@adamhjk](https://twitter.com/adamhjk)
* The <b>Community Manager</b> guides and meets regularly with community advocates, helps enforce corrective actions, hears appeals, is responsible for maintaining a list of incidents, and ensures pertinent information is shared with necessary parties. The Community Manager is [Ian Henry]([email protected]), [@eeyun___](https://twitter.com/eeyun___).
Copy link

Choose a reason for hiding this comment

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

Maybe have a separate section on sharing of information. what, and who. I know it's mentioned later but calling it out explicitly is good.

I imagine it's something like:

  • Repo of incidents - community organizers
  • acknowledgement of report, and any followup - person reporting
  • set of corrective actions and consequences of failing to follow through - person being reported

There also feels like an explicit need to say that that folks who are community organizers that are removed from the role will upon removal eliminate any local copies of the repo that they have in addition to the having role revocated with github.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh yeah thats totally worth noting. Hadn't thought about local copies.

Can you clarify what you mean about sharing of information?

Copy link

Choose a reason for hiding this comment

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

In the Community Manager role it says "ensures pertinent information is shared with necessary parties" .. I'm suggesting having a specific section that documents the "Sharing of Information" and what it means. I think many times one of the challenges of folks to come forward with incidents is fear and how the information about what happened and who they are will get used for retaliation. So when you have something that says "pertinent information is shared with necessary parties" questions come up.. what is pertinent information, who has access to it, etc.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Perfect that's excellent!

Signed-off-by: Ian Henry <[email protected]>

punctuation and typos

Signed-off-by: Ian Henry <[email protected]>

Code of Conduct New Iteration

Signed-off-by: eeyun <[email protected]>

Updates to Code of Conduct to be more explicit on acceptability

Signed-off-by: Ian Henry <[email protected]>

corrected ben's email address

Signed-off-by: eeyun <[email protected]>
@eeyun eeyun force-pushed the eeyun/code_of_conduct branch from cc32463 to 019e975 Compare May 31, 2017 22:23
@eeyun
Copy link
Contributor Author

eeyun commented May 31, 2017

As no further responses have popped up to this PR. I think it's ready to be merged! As we are still a relatively young community there should be a bit of flexibility when it comes to altering the CoC moving forward but for now this is a solid starting point!

@eeyun eeyun merged commit ce204ba into master May 31, 2017
@eeyun eeyun deleted the eeyun/code_of_conduct branch August 3, 2017 22:40
@christophermaier christophermaier added Documentation Flags an issue / PR for attention by the technical documentation team and removed A-documentation labels Aug 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Flags an issue / PR for attention by the technical documentation team
Projects
None yet
Development

Successfully merging this pull request may close these issues.