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

Status of GitHub LTS team #84

Closed
Trott opened this issue Mar 6, 2018 · 8 comments
Closed

Status of GitHub LTS team #84

Trott opened this issue Mar 6, 2018 · 8 comments

Comments

@Trott
Copy link
Member

Trott commented Mar 6, 2018

Is the @nodejs/LTS team now supplanted by and/or absorbed by the @nodejs/Release team, similar to the way @nodejs/CTC is now supplanted by and/or absorbed by @nodejs/TSC?

Trying to figure out if LTS should be made a subteam of members or not.

@MylesBorins
Copy link
Contributor

MylesBorins commented Mar 6, 2018 via email

@Trott
Copy link
Member Author

Trott commented Mar 6, 2018

@MylesBorins OK, thanks. I'll make LTS a subteam of members for now. If it gets moved to be a subteam of Release, then it will be a sub-subteam of members, which is also fine.

@Trott Trott closed this as completed Mar 6, 2018
@Trott
Copy link
Member Author

Trott commented Mar 6, 2018

(I'm guessing here, but you probably don't want LTS to be a subteam of Release. If you do that, then any @-mentioning of Release about release-related things will also @-mention LTS. And any access you give Release will also be granted to LTS. Permissions and access flow down to subteams.)

@MylesBorins
Copy link
Contributor

MylesBorins commented Mar 6, 2018 via email

@Trott
Copy link
Member Author

Trott commented Mar 6, 2018

@MylesBorins Hmmm.... The description for @nodejs/release is "Access to cut Node.js releases". Either the description wrong or @nodejs/release is not the Release WG but instead the releasers.

@gibfahn
Copy link
Member

gibfahn commented Mar 6, 2018

So we discussed this a lot previously, and decided to have:

  • nodejs/Release, with subteams:
    • nodejs/LTS
    • nodejs/releasers (what is now nodejs/release)
    • nodejs/citgm

It's all documented in the readme, and was discussed exhaustively in nodejs/Release#223.

Unfortunately no-one actually did the renaming.

I think the crucial thing is the updating of CI to change access permissions for nodejs/release to nodejs/releasers. Other than that there shouldn't be anything that needs particular care.

@mhdawson
Copy link
Member

mhdawson commented Mar 7, 2018

@gibfahn are you going to handle the updates?

@gibfahn
Copy link
Member

gibfahn commented Mar 9, 2018

I guess I can do that, sure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants