Skip to content

github-team-discussion policy#161

Closed
refack wants to merge 7 commits intomasterfrom
github-team-discussion
Closed

github-team-discussion policy#161
refack wants to merge 7 commits intomasterfrom
github-team-discussion

Conversation

@refack
Copy link
Copy Markdown
Contributor

@refack refack commented Jun 14, 2018

An alternative to #121
For approval by the @nodejs/tsc and @nodejs/community-committee

As discussed in the 2018-06-14 meeting of the Community Committee, we recommend considering "Team Discussions" as public, and setting that as the default organizational policy. If chartered sub projects or working groups want to set their own rules, it will be their responsibility to enforce those rules and moderate those discussions.

@refack refack added the tsc-review Issues and PRs that need attention or reviews from the TSC. label Jun 14, 2018
@refack refack self-assigned this Jun 14, 2018
@refack refack requested review from a team June 14, 2018 17:14
richardlau

This comment was marked as off-topic.

MylesBorins

This comment was marked as off-topic.

@refack
Copy link
Copy Markdown
Contributor Author

refack commented Jun 14, 2018

I don't think that we should be encouraging using discussions. Discussions are not public. They may be "publicly" viewable to the org, but that isn't open / transparent to the greater community.

I get what you are saying... What I was trying to phrase was a statement that does not
encourage, but makes some statement on their use, i.e. they are not to be used as a private forum.

There are some instances where it makes sense to use discussions, such as checking for objections on nominations. If we are to document anything for discussions I believe it should be listing the few times where it is appropriate to use them.

So that's where the comm-comm wants to delegate this decision to nodejs/node for example. To find if there is consensus about this use (I don't think there is Ref: nodejs/node#17162)

@refack
Copy link
Copy Markdown
Contributor Author

refack commented Jun 14, 2018

@MylesBorins PTAL if I was able to phrase this better.

@bnb
Copy link
Copy Markdown
Contributor

bnb commented Oct 5, 2018

As discussed in the 2018-06-14 meeting of the Community Committee, we recommend considering "Team Discussions" as public

Can you please share the reference?

So that's where the comm-comm wants to delegate this decision to nodejs/node for example

Again, can you please provide an exact reference?

@jakeNiemiec
Copy link
Copy Markdown

@hackygolucky
Copy link
Copy Markdown
Contributor

hackygolucky commented Oct 5, 2018

I'm not hearing anything in this recording that settles on us writing a new policy for admin. In this recording, I distinctly recommended -not- putting this in the admin repo and that it should be up to the CommComm or the TSC to first set this and then both groups, if this works, could both enact this as a guideline.

I think we should also focus on guidelines that recommend where TO have conversations instead of where not to do things. CommComm isn't heavily using Team Discussions right now, so I'm wondering what instigated this in the first place? We try not to write rules and optimize for things that haven't happened.

In short, recommendations:

  1. Move this PR to CommComm
  2. Edit this PR for recommending guideline around public discourse(if you would like to explicitly capture transparency amongst the other values, you can reference them from our charter and membership expectations) instead of where NOT to do things.

@refack
Copy link
Copy Markdown
Contributor Author

refack commented Nov 18, 2018

ping @nodejs/community-committee @nodejs/tsc
IMHO we really should have some wording around this gray-area feature. If not the wording suggested in this PR then something else...

Comment thread GITHUB_ORG_MANGEMENT_POLICY.md Outdated
Comment thread GITHUB_ORG_MANGEMENT_POLICY.md Outdated
Comment thread GITHUB_ORG_MANGEMENT_POLICY.md Outdated
Co-Authored-By: refack <refack@gmail.com>

*For General consideration*: The current implementation of GitHub's "Team
Discussions" is incongruent with the organization's policy. The maximal
visibility of "public" threads is limited by GitHub to members of the org only,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
visibility of "public" threads is limited by GitHub to members of the org only,
visibility of public threads is limited by GitHub to members of the org only,

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

But here it's referring to a name of a feature. Not the general idea...

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

i.e. this:
image

*For General consideration*: The current implementation of GitHub's "Team
Discussions" is incongruent with the organization's policy. The maximal
visibility of "public" threads is limited by GitHub to members of the org only,
so they are not what the organization defines as public. Use of the "private"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
so they are not what the organization defines as public. Use of the "private"
so they are not what the organization defines as public. Use of the private

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ditto

@refack
Copy link
Copy Markdown
Contributor Author

refack commented Nov 18, 2018

@hackygolucky I feel that you last comment is slightly off topic, and IMHO it's not empathetic to the opinions of the community member to whom you are apparently answering. I ask to to edit it, or remove it altogether.

@refack refack dismissed MylesBorins’s stale review November 18, 2018 03:32

Stale review. Issue was addressed, and reviewer has not responded to a request to re-review.

@refack refack removed the request for review from a team November 18, 2018 03:33
@jasnell
Copy link
Copy Markdown
Member

jasnell commented Nov 18, 2018

@refack... To which comment of @hackygolucky's are you referring to? I don't see any that are off topic or inappropriate in any way, let alone any that should be moderated or deleted.

@nodejs/moderation... I'd like to challenge this moderation request. It would get good to get input from the full moderation team on whether any action is necessary here.

Copy link
Copy Markdown
Member

@jasnell jasnell left a comment

Choose a reason for hiding this comment

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

I'm not convinced this is something we need to have a policy statement for at this time.

@othiym23
Copy link
Copy Markdown

@nodejs/moderation... I'd like to challenge this moderation request. It would get good to get input from the full moderation team on whether any action is necessary here.

For the record, even though @hackygolucky is a member of the moderation team, she's recusing herself from this discussion.

@jasnell As you've requested, I've reviewed @hackygolucky's comment and @refack's response, and see only a request for clarification and some recommendations from @hackygolucky, which seem well within the scope of @hackygolucky's role as a member of commcomm. I don't see unwelcoming language or anything else that would fall under the purview of moderation.

@refack
Copy link
Copy Markdown
Contributor Author

refack commented Nov 18, 2018

I specifically addressed @hackygolucky personally, when asking her to reconsider, since I did not consider this a moderation issue.

I prefer to explain my position further in a non-public forum.

@refack
Copy link
Copy Markdown
Contributor Author

refack commented Nov 18, 2018

I'm not convinced this is something we need to have a policy statement for at this time.

@jasnell policy was requested by several community members. I don't have a strong preference to the specifics, but I do think we should have something. There's #121 as an alternative.

@Trott
Copy link
Copy Markdown
Member

Trott commented Apr 23, 2019

There seems to be sufficient opposition to this that consensus is unlikely. I'm going to close this.

@refack and anyone else: Feel free to re-open (or comment if GitHub doesn't permit you to re-open) if you disagree with my assessment. Closing this is me making my best assessment as to the right thing to do with this PR at this time, but that's just my opinion only.

@Trott Trott closed this Apr 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tsc-review Issues and PRs that need attention or reviews from the TSC.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants