Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Add process for handling reports #271
Add process for handling reports #271
Changes from 8 commits
de67a3e
3a41881
0fe421d
fbbf763
e3bf6ef
dac1d84
b9b2131
25d3fd2
e996830
516f586
b3f81e8
16e4431
3e0d1a9
f547434
1d5aaf2
2eef5ce
98902f7
bd5837b
366444e
9671542
377b53e
6d12e32
37c2999
d475032
84e9406
f575cec
d84e0d3
a58aa13
20ea625
d72c623
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
similar to above. This document should outline the actual process rather than questions about process.
Also, is this document for project maintainers or external individuals? That would likely change the process a bit.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This document was written initially for project maintainers.
Removing content to README.md. Thanks for calling out.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are we sure we want to commit to this? If we want to make even anonymized summaries of reports / actions taken that will not be possible with the current reading.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Link to channels here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Link to roles?
Also
right
seems like an odd way to frame it. perhaps "On call staff is alerted"There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/Facts and opinions are gathered/Information gathering
We won't know what are facts or opinions... seems odd to frame it this way
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/Facts are discussed/Information is discussed
We don't know what are "facts"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we link to the section?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure I'm 100% in support of this. While the reportee should be consulted in deciding the resolution they should not necessarily be in the position to make this decision.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggested revision:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would this section be an appropriate place to also define "reporter" and "reportee". I realise now that I'd missed earlier the rather significant difference between them.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is an excellent opportunity to clarify language.
I had used the word "reportee" to refer to the person/s against whom a violation was committed.
I used the word "reporter" when the person who made the report is not the same as the person against whom the violation was committed.
When we clarify, I can go in and change to make the language consistent, it may not be.
We also need a word to refer to the person/s who committed the violation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will there be any particular rule to decide on who will be this person? Like they cannot be from the same project, just to avoid any conflict of interest.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's a good issue to raise. We should clarify. For example, my own opinion is that this person should currently not be directly involved with a specific project (OpenJS Foundation is fine), but past history is ok.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These questions should be moved to the readme