The Wayback Machine - https://web.archive.org/web/20201214101454/https://github.com/nodejs/admin/issues/517
Skip to content
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 collaborators as "Read" to this repo #517

Open
MylesBorins opened this issue Jun 24, 2020 · 10 comments
Open

Add collaborators as "Read" to this repo #517

MylesBorins opened this issue Jun 24, 2020 · 10 comments

Comments

@MylesBorins
Copy link
Member

@MylesBorins MylesBorins commented Jun 24, 2020

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

@tniessen
Copy link
Member

@tniessen tniessen commented Jun 28, 2020

@codeekage
Copy link
Contributor

@codeekage codeekage commented Jun 28, 2020

+1

@tniessen
Copy link
Member

@tniessen tniessen commented Jun 28, 2020

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

Citing @bnb from #516 (comment):

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

GitHub seems to confirm this.

@mmarchini
Copy link
Member

@mmarchini mmarchini commented Jun 29, 2020

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

@gireeshpunathil
Copy link
Member

@gireeshpunathil gireeshpunathil commented Jun 29, 2020

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)

@bnb
Copy link
Member

@bnb bnb commented Jun 29, 2020

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

This would exclude CommComm from the discussion.

@tniessen
Copy link
Member

@tniessen tniessen commented Jun 29, 2020

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

@gireeshpunathil
Copy link
Member

@gireeshpunathil gireeshpunathil commented Jun 29, 2020

This would exclude CommComm from the discussion.

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

@bnb
Copy link
Member

@bnb bnb commented Jun 29, 2020

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

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

@mmarchini
Copy link
Member

@mmarchini mmarchini commented Jun 29, 2020

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
6 participants
You can’t perform that action at this time.