admin icon indicating copy to clipboard operation
admin copied to clipboard

Add collaborators as "Read" to this repo

Open MylesBorins opened this issue 4 years ago • 10 comments

@nodejs/collaborators or @nodejs/members should be added so they can participate in locked conversations.

MylesBorins avatar Jun 24 '20 18:06 MylesBorins

+1 to @nodejs/members.

tniessen avatar Jun 28 '20 14:06 tniessen

+1

codeekage avatar Jun 28 '20 15:06 codeekage

Adding @nodejs/members with "Read" access might not help:

Citing @bnb from https://github.com/nodejs/admin/issues/516#issuecomment-650473171:

only people with write access to this repository can now comment, which excludes much of the project's collaborators.

GitHub seems to confirm this.

tniessen avatar Jun 28 '20 17:06 tniessen

Any concerns on adding @nodejs/members with write permission? If we want to limit commits here, we could set branch restrictions.

mmarchini avatar Jun 29 '20 02:06 mmarchini

I would rather move the relevant issue ( #516) to a less restricted repo such as core. Adding write permissions to the org members does not seem to align well with the stated purpose of this repo? (Administration for TSC + Community Committee collaborative efforts)

gireeshpunathil avatar Jun 29 '20 02:06 gireeshpunathil

I would rather move the relevant issue ( #516) to a less restricted repo such as core.

This would exclude CommComm from the discussion.

bnb avatar Jun 29 '20 15:06 bnb

We should probably just unlock #516 then, and delete undesired comments quickly.

tniessen avatar Jun 29 '20 15:06 tniessen

This would exclude CommComm from the discussion.

apologies, did not know that, and definitely did not mean it!

gireeshpunathil avatar Jun 29 '20 16:06 gireeshpunathil

apologies, did not know that, and definitely did not mean it!

no offense taken! Just wanted to make sure that it was shared <3

bnb avatar Jun 29 '20 20:06 bnb

I agree with @tniessen, and if the volume of undesired comments grows quickly, we can lock again and figure out another way to handle this.

mmarchini avatar Jun 29 '20 20:06 mmarchini