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

remove bnoordhuis from lists #67

Merged
merged 1 commit into from
Sep 28, 2017
Merged

remove bnoordhuis from lists #67

merged 1 commit into from
Sep 28, 2017

Conversation

Trott
Copy link
Member

@Trott Trott commented Sep 27, 2017

Ben has resigned from TSC, so removing from the TSC and CTC lists. I
made the judgment call that he should be removed from the security list
as well as everyone else on that list is TSC and they can always loop
him in if need be. Not feeling too strongly about that though if there's
strong opposition.

@Trott
Copy link
Member Author

Trott commented Sep 27, 2017

This should probably land pretty quickly as it doesn't make a whole lot of sense to take a few days to update email lists when people resign and stuff. (And I don't think it makes sense to delay landing this based on any discussion around security@ email because it would be trivial to re-add him if that ends up being the consensus.)

MylesBorins

This comment was marked as off-topic.

@evanlucas
Copy link

Should him resigning from the TSC also remove him from security? I think he has generally been one of the first responders to security issues.

@bnoordhuis
Copy link
Member

I'll still be around as a reviewer and contributor (and security triager), just not as a TSC member.

Ben has resigned from TSC, so removing from the TSC and CTC lists. I
made the judgment call that he should be removed from the security list
as well as everyone else on that list is TSC and they can always loop
him in if need be. Not feeling too strongly about that though if there's
strong opposition.
@Trott
Copy link
Member Author

Trott commented Sep 27, 2017

Added Ben back to the security@ email list. @MylesBorins PTAL

MylesBorins

This comment was marked as off-topic.

evanlucas

This comment was marked as off-topic.

@rvagg rvagg merged commit 9928a77 into nodejs:master Sep 28, 2017
@rvagg
Copy link
Member

rvagg commented Sep 28, 2017

shipped

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

Successfully merging this pull request may close these issues.

5 participants