Skip to content

Conversation

mkoeppe
Copy link
Contributor

@mkoeppe mkoeppe commented Dec 9, 2023

We add examples to the Code of Conduct that explain that even well-intended conduct can be problematic.

Related sage-devel discussions:

📝 Checklist

  • The title is concise, informative, and self-explanatory.
  • The description explains in detail what this PR is about.
  • I have linked a relevant issue or discussion.
  • I have created tests covering the changes.
  • I have updated the documentation accordingly.

⌛ Dependencies

@williamstein
Copy link
Contributor

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.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Dec 30, 2023

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.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Feb 14, 2024

@jhpalmieri
Copy link
Member

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.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Feb 14, 2024

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.

@jhpalmieri
Copy link
Member

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.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Feb 14, 2024

In any case, I think that these proposed changes need buy-in [...]

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.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Feb 14, 2024

This is a "code of conduct," not a "code of conduct response and enforcement manual." The two have different purposes and different audiences.

Making them separate documents can make sense, but not because one should be public and the other should not.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Feb 14, 2024

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.

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.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Feb 14, 2024

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.

Copy link

Documentation preview for this PR (built with commit e1f873a; changes) is ready! 🎉

@jhpalmieri
Copy link
Member

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)?

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Feb 28, 2024

Either way is fine with me

@jhpalmieri
Copy link
Member

See #37501.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Feb 28, 2024

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

vbraun pushed a commit to vbraun/sage that referenced this pull request Mar 30, 2024
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
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.

3 participants