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

chore: add vote data for automatic collaborator move-to-emeritus #1139

Closed
wants to merge 1 commit into from

Conversation

Trott
Copy link
Member

@Trott Trott commented Dec 15, 2021

Copy link
Member

@mhdawson mhdawson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@cjihrig cjihrig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was not clear to me that this was a vote. Apparently I wasn't the only one.

@Trott
Copy link
Member Author

Trott commented Dec 16, 2021

It was not clear to me that this was a vote. Apparently I wasn't the only one.

It wasn't officially, but I figure that erring on the side of considering "waiting for a majority of TSC to approve" type things TSC votes is preferable. The TSC Charter puts the baseline for TSC participation as participating in one vote in a three month period, so the more things that are considered votes, the easier it is to meet that requirement. Non-participation in a vote is not penalized, so counting more things as votes doesn't hurt anyone. That's not the case for meetings, where there is a percentage attendance threshold.

If you still think this shouldn't land, I won't argue and will close it.

@cjihrig
Copy link
Contributor

cjihrig commented Dec 16, 2021

Non-participation in a vote is not penalized, so counting more things as votes doesn't hurt anyone.

I think it's a very unlikely scenario, but couldn't this technically become the only vote in a 3 month period?

I won't block this, but I really think we should be more explicit moving forward about what constitutes a vote since it can technically impact someone.

@Trott
Copy link
Member Author

Trott commented Dec 16, 2021

I think it's a very unlikely scenario, but couldn't this technically become the only vote in a 3 month period?

Sure but in that case, leaving this out so that there are no votes in a 3-month period would mean that only attendance at meetings matters. Adding a vote in would make it easier to meet the either/or requirement of vote or attendance. With no votes, you have to be attending the meetings or else you don't meet the requirements.

@Trott
Copy link
Member Author

Trott commented Dec 16, 2021

I won't block this,

I'm going to close this. It wasn't a vote strictly speaking and I don't want to do something that can be questioned/debated if there's a -0 in the mix.

but I really think we should be more explicit moving forward about what constitutes a vote

I can't argue with that.

@Trott Trott closed this Dec 16, 2021
@Trott Trott deleted the the-world-in-crisis branch December 16, 2021 05:12
@targos
Copy link
Member

targos commented Dec 16, 2021

I assumed a change to the governance document required a vote but I'm probably wrong

@Trott
Copy link
Member Author

Trott commented Dec 16, 2021

I assumed a change to the governance document required a vote but I'm probably wrong

The usual consensus process is sufficient. We can't change the TSC charter that way, but governance of the nodejs repo is ordinary decision-making.

We could choose to either informally agree or formally require that such changes go through a TSC vote.

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.

None yet

5 participants