Skip to content
This repository has been archived by the owner on Sep 2, 2023. It is now read-only.

governance: be explicit about meetings new members can join #218

Merged
merged 1 commit into from
Nov 21, 2018

Conversation

MylesBorins
Copy link
Contributor

Be explicit that if a new members has requested to join and the issue has no objections that can attend the meeting where their membership is approved

@MylesBorins MylesBorins added the modules-agenda To be discussed in a meeting label Nov 7, 2018
Copy link
Contributor

@sendilkumarn sendilkumarn left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

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

LGTM

@bmacnaughton
Copy link

bmacnaughton commented Nov 8, 2018

Thanks for working on clarifying this. I didn't want to presume the right to show up.

I'm completely outside the process, so I might be missing some assumptions. This change sounds like it is focused on new members where the membership approval has largely been determined prior to the meeting making it official. I would not have been sure that this applied to observers had I read it before the meeting.

If the intention is that members means members and observers then it should probably be explicit. I can deduce that's what it means because observers already can attend meetings, but maybe it should say observers instead of members?

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

@MylesBorins MylesBorins merged commit 04aca82 into nodejs:master Nov 21, 2018
@GeoffreyBooth GeoffreyBooth removed modules-agenda To be discussed in a meeting labels May 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.