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

Change language as defined by NF Legal #384

Merged
merged 1 commit into from
Jan 10, 2019

Conversation

bnb
Copy link
Contributor

@bnb bnb commented Sep 25, 2018

Original PR contents:

This change has been approved at the Board level but was simplifed from changes we'd proposed months ago in THIS PR. Due to the leaving of the executive director and a misunderstanding in how we have historically processed charter changes from the CommComm or the TSC to the Board, this took some time to weed out language vs. changes for things such as CC --> CommComm.

This provides the minimal language changed as defined by Node Foundation Legal in order to make the CommComm Director codified in the charter but as mentioned, was originally provided by CommComm to the Board as similar language.

**it has been suggested for future PRs that we make sure to tease out important governance changes from copy changes for clarity's sake.

Original PR author: @hackygolucky

Original PR link (reverted for further discussion): #380

author: @hackygolucky 

Original PR (reverted for further discussion): #380
@refack
Copy link
Contributor

refack commented Sep 25, 2018

FTR: I don't have a proposed change. I just wanted to make sure we have all the information, and that the changes that do land are those that have been approved.

What I did notice is that the current change (addition of 7.5) is self conflicting, and seemingly repetitive

7.4

The CommComm will elect from amongst voting CommComm members a CommComm Chairperson to work on building an agenda for CommComm meetings and collaborate with the Individual Membership Directors the wishes of the CommComm to the Board for a term of one year according to the Node.js Foundation’s By-laws. The CommComm shall hold annual elections to select a CommComm Chairperson; there are no limits on the number of terms a CommComm Chairperson may serve.

7.5

The CommComm will elect from amongst voting CommComm members:

  • a CommComm Chairperson, responsible for preparing agendas and coordinating CommComm meetings,
  • and
    a representative to serve on the Node.js Foundation Board of Directors pursuant to its Bylaws.
  • Each person shall serve a one year term, and may serve for more than one term. The same person may not hold both positions at the same time.

I wanted to make sure if the final approved language was to add paragraph (5) or to replace paragraph (4).

@MylesBorins
Copy link
Contributor

@refack what was approved by the board was this exact language. Keep in mind that the "true charter" is an amalgamation of all the motions approved by the board. We could in theory go through all those motions and redraft a charter and have that approved. So it is important to think about the "intent" or "spirit" of the charter. That does not mean we should not fix this, more that it isn't a huge deal to have it temporarily not make as much sense.

I've opened #385 to simplify the text, this is based on the current text found in the TSC charter. Notice that it does remove the bit regarding the individual member director /cc @williamkapke

@refack

This comment has been minimized.

@refack
Copy link
Contributor

refack commented Sep 25, 2018

To make myself clear. Since this change was authored and coordinated by @hackygolucky I would like their review and approval.

@MylesBorins MylesBorins requested a review from a team September 25, 2018 20:18
@MylesBorins
Copy link
Contributor

@refack the original post by @bnb assigned @hackygolucky as a reviewer on this

@hackygolucky
Copy link
Contributor

I'm not sure what we want here as I was the original PR post, so I don't think this makes sense for me to +1? The point of process is to get two +1s from CommComm members, mark as DO NOT MERGE, and then the Board approves what we've already approved. Once that occurs. It is ready for merge. I can merge, I shouldn't +1 my own PR.

amiller-gh

This comment was marked as off-topic.

@bnb
Copy link
Contributor Author

bnb commented Sep 26, 2018

I can't technically add a +1 with a GitHub approval since I created this PR, but consider this my +1.

mhdawson

This comment was marked as off-topic.

WaleedAshraf

This comment was marked as off-topic.

@@ -24,7 +24,7 @@ Changes to CommComm membership should be posted in the agenda, and may be sugges

No more than one-fourth of the CommComm members may be affiliated with the same employer or leadership of a community/ecosystem organization. If removal or resignation of a CommComm member, or a change of employment by a CommComm member, creates a situation where more than one-fourth of the CommComm membership shares an employer, then the situation must be immediately remedied by the resignation or removal of one or more CommComm members affiliated with the over-represented employer(s).

The CommComm members shall consist of active members of Community Projects and the two Individual Membership Directors as defined in Section 10.
The CommComm members shall consist of active members of Community Projects and the two Individual Membership Directors.
Copy link
Contributor

Choose a reason for hiding this comment

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

Not two anymore

Copy link
Member

Choose a reason for hiding this comment

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

Lets make this change on #385 👍

@amiller-gh
Copy link
Member

This has been open for a while with 7/15 giving explicit approval, and no explicit opposition – we're going to have Tracy bring this to the board this week for approval and then we can merge it in 🎉

Please pile on to #385 to make further language changes. Given our velocity the past week, I don't think we'll have that PR ready to present by this week's board meeting, but I'd like to prioritize it so we can bring it to the next one for approval. Go forth and govern 😉

@joesepi
Copy link
Member

joesepi commented Nov 1, 2018

LGTM

@hackygolucky
Copy link
Contributor

Board approved. Removed do-not-merge tag.

@hackygolucky hackygolucky merged commit beac53d into master Jan 10, 2019
@keywordnew keywordnew deleted the bnb/change-language-as-defined-by-nf-legal branch January 9, 2020 17:44
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.