-
-
Notifications
You must be signed in to change notification settings - Fork 660
CODE_OF_CONDUCT.md
: Add examples of problematic conduct, describe sage-abuse committee's procedures and sanctions
#36844
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
Conversation
I think it would be useful to have a proposal about consequences for violating the code of conduct, and how to decide whether or not something is a violation. For deciding, sage-abuse could have an internal vote. For consequences, they could include deleting the relevant messages, lowering or removal of privileges, etc. This was never formalized. |
I agree that this would be very useful. As a first draft, we could just add a list of types of sanctions that have been used in the past 10 years, if you are able to provide this. Going forward, there probably also should be a description of an appointment procedure for the sage-abuse committee. |
@roed314 Examples of CoC with clear enforcement protocols: |
I don't think that proposed lines 72-86 in CODE_OF_CONDUCT.md belong there. They are not useful for the general Sage community, only for members of the Sage abuse committee. They read like a critique of the functioning of that committee, perhaps deserved, but they don't belong in this document. |
Have you looked at the link https://numfocus.org/code-of-conduct/response-and-enforcement-events-meetups or the other links I shared? Spelling out such details is very much part of the best practices for CoC enforcement. |
This is a "code of conduct," not a "code of conduct response and enforcement manual." The two have different purposes and different audiences. Anyway, I did take a quick look, and my impression is that if there is such content, it is much less than half of the material on the possible responses. In my opinion the current draft puts too much emphasis on the analysis of the responses (with a critical tone, in my opinion) rather than the responses themselves. I don't see language with the same tone in the numfocus document that you linked. I may very well be missing it, but that's because the bulk of that document doesn't have that tone or that content or that intent. I think that language like the section on "Response and Potential Consequences" in https://numfocus.org/code-of-conduct would be more suitable for this document. (By the way, if we just wanted to, for example, adopt the numfocus response and enforcement manual, I would have no objections. Maybe it needs modification to address online interactions, since it is focused on in-person events.) In any case, I think that these proposed changes need buy-in from someone like William or David R, since they establish policies and a tone related to the Sage abuse committee. |
Of course! All of such changes need writing and editing, discussion, ratification, etc. and obviously cannot be put in place by just the normal PR review process. |
Making them separate documents can make sense, but not because one should be public and the other should not. |
Yes, the NumFOCUS one does have the focus on in-person interactions. I shared other links too, though. The SciPy one leads you to https://docs.scipy.org/doc/scipy/dev/conduct/report_handling_manual.html The process of "adopting" a prefab manual will certainly have to involve more than just to declare that it is now in force. It will require reflection on how what is described relates to past practices. |
Documenting the types of sanctions that the committee has used in the past (as suggested above #36844 (comment) in response to William's comment) serves the purpose of making it known to the community that enforcement happens and is not just a "theoretical possibility". It can also document the continuity of the committee's mandate. Including an analysis of what specific possible sanctions do -- and don't do -- as part of the resolution process may be helpful for future enforcement. Right now in this early draft, these two things (past sanctions, possible sanctions) happen in the same place. There's no need for this. I'll rewrite it. |
Documentation preview for this PR (built with commit e1f873a; changes) is ready! 🎉 |
I have a "competing" proposal for changes to the code of conduct, along with a new document about how to handle violations, based heavily on SciPy's similar document. Should I open a separate PR or push changes here (in place of yours, not on top of them)? |
Either way is fine with me |
See #37501. |
Closing in favor of #37501. @jhpalmieri I still think including examples of conduct that is intended to be helpful but can be offensive to others would be valuable |
sagemathgh-37501: Code of conduct changes - **Revise the code of conduct** <!-- ^ Please provide a concise and informative title. --> <!-- ^ Don't put issue numbers in the title, do this in the PR description below. --> <!-- ^ For example, instead of "Fixes sagemath#12345" use "Introduce new method to calculate 1 + 2". --> <!-- v Describe your changes below in detail. --> <!-- v Why is this change required? What problem does it solve? --> <!-- v If this PR resolves an open issue, please link to it here. For example, "Fixes sagemath#12345". --> Revise the code of conduct: add a diversity component and sections on reporting guidelines and how reports are handled. Also add a new document about the role of the Sage Code of Conduct Committee. The changes to the first document are based on similar documents from SciPy and NumFOCUS, plus inspiration and some language from the changes proposed at sagemath#36844. The second document is heavily based on the corresponding document from SciPy. Based on top of sagemath#37054. ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> - [X] The title is concise and informative. - [X] The description explains in detail what this PR is about. - [X] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation accordingly. ### ⌛ Dependencies <!-- List all open PRs that this PR logically depends on. For example, --> <!-- - sagemath#12345: short description why this is a dependency --> sagemath#37054 URL: sagemath#37501 Reported by: John H. Palmieri Reviewer(s): Dima Pasechnik, John H. Palmieri, Kwankyu Lee, Matthias Köppe, nbruin, roed314, Tobias Diez
We add examples to the Code of Conduct that explain that even well-intended conduct can be problematic.
Related sage-devel discussions:
📝 Checklist
⌛ Dependencies