-
Notifications
You must be signed in to change notification settings - Fork 569
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
Grant release team maintainer access to release-related tooling #557
Comments
FWIW There's a shadow @nodejs/citgm-admins team that is used to control edit access to the CITGM CI jobs. We can probably keep the name of that group as-is. |
+1
…On Thu, Mar 26, 2020 at 1:15 PM Richard Lau ***@***.***> wrote:
FWIW There's a shadow @nodejs/citgm-admins
<https://github.com/orgs/nodejs/teams/citgm-admins> team that is used to
control edit access to the CITGM CI jobs. We can probably keep the name of
that group as-is.
—
You are receiving this because you are on a team that was mentioned.
Reply to this email directly, view it on GitHub
<#557 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AADZYV4YXLDGX6DW2KSMWVTRJOESXANCNFSM4LUMUBEA>
.
|
+1 |
2 similar comments
+1 |
+1 |
BTW I opened nodejs/admin#488 |
If we don't want to move forward with this work we should probably close this issue |
The last few times this issue has come up in Release WG meetings the consensus has been to add @nodejs/releasers to these repositories rather than creating a new team or repurposing the citgm team. |
release-automation
This was discussed in #735. We no longer plan to rename the CITGM team. Instead, this issue can be closed once the releaser have been granted maintainer access to the mentioned repositories. I've updated the issue description accordingly. |
Given no objections and a 👍 I've added the releasers team to https://github.com/nodejs/branch-diff and https://github.com/nodejs/changelog-maker. |
Opened nodejs/node-core-utils#628 for node-core-utils. |
Based on nodejs/citgm#901 (comment) I've added citgm to the list. I'll open an issue for it in the citgm repository. |
Should https://github.com/nodejs/commit-stream be added to this list of repos? |
Added |
#547 opened further questions about the structure of the Release WG. It has been discussed that release team should have some responsibility over release related tooling.
@nodejs/releasers
write access to release related toolingchangelog-maker
commit-stream
branch-diff
node-core-utils
citgm
The text was updated successfully, but these errors were encountered: