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 #380

Merged
merged 1 commit into from
Sep 25, 2018
Merged

Change language as defined by NF Legal #380

merged 1 commit into from
Sep 25, 2018

Conversation

hackygolucky
Copy link
Contributor

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.

Change language as defined by NF Legal but originally provided by CommComm to Board
Bamieh

This comment was marked as off-topic.

WaleedAshraf

This comment was marked as off-topic.

refack

This comment was marked as off-topic.

refack

This comment was marked as off-topic.

refack

This comment was marked as off-topic.

@bnb
Copy link
Contributor

bnb commented Sep 24, 2018

Just wanted to note:

We've been waiting for this to come back since April. I deeply appreciate the work that @hackygolucky has done to make it happen.

My advice is that we merge it as-is, then bring any changes/suggestions/updates in a different PR so that we can actually land this rather than stalling it out. We should certainly be more liberal about updating this doc, but we should get a solid and common base under our feet before we begin that 💖

bnb

This comment was marked as off-topic.

mhdawson

This comment was marked as off-topic.

@bnb
Copy link
Contributor

bnb commented Sep 25, 2018

Given this is four days old, and there are no -1s, I'm going to merge it. Thank you @hackygolucky, @MylesBorins, @chowdhurian, and the rest of the CommComm for the amount of work that y'all put into making this happen.

We should talk about the impact of this in the next CommComm meeting.

@bnb bnb merged commit fa78026 into master Sep 25, 2018
@refack
Copy link
Contributor

refack commented Sep 25, 2018

I think the merge was premature since my request for clarification was not addressed. So ATM our charter states:

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.

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.

refack added a commit that referenced this pull request Sep 25, 2018
This reverts commit fa78026, reversing
changes made to b8e6b49.

Due to premature merge
@MylesBorins

This comment has been minimized.

@MylesBorins
Copy link
Contributor

MylesBorins commented Sep 25, 2018

@refack fwiw, this should land and a new PR to fix it should be made and that text approved by board. Since this specific change was approved you are way better off landing it and then iterating imho

edit: to be very explicit. Every change, the exact wording, needs to be approved by the board and legal. As this change was approved and makes the directorship official we should land it as is. Changes to this text should be made in a new PR and the CommComm should build consensus around that text, which can then be sent to the board + legal for approval.

@refack
Copy link
Contributor

refack commented Sep 25, 2018

@refack fwiw, this should land and a new PR to fix it should be made and that text approved by board. Since this specific change was approved you are way better off landing it and then iterating imho

Since I don't have reference to the Board's decisions, I wanted to make sure whether the approved change was an addition of a new paragraph or a replacement paragraph.

Where can I find the minutes of the board's meeting?

@jakeNiemiec
Copy link

jakeNiemiec commented Sep 25, 2018

@refack I think it was this one: https://youtu.be/r1WY6v00NhM

But, It looks like they explicitly say that "the individual membership and director stuff" would need to come back to the board before merging: https://youtu.be/r1WY6v00NhM?t=6m54s

Edit: This was not intended as a "gotcha", I have nothing but respect for (and work done by) Tracy.

@Tiriel
Copy link
Contributor

Tiriel commented Sep 25, 2018

I'm quite not comfortable with what's happening here.
I don't want to start arguing over semantics but since there was no hard -1, there was legitimacy to land this PR. And I think we should settle down here and take some time because I worry we have some bad steam building and escalation.

I'd like to request this is talked about in the next meeting if it's ok for everyone. I some wish it to be private, I'd even second it but if you all deem it not necessary, fine by me.

@refack
Copy link
Contributor

refack commented Sep 25, 2018

I'm quite not comfortable with what's happening here.

I'm sorry about that, it is not my intention to cause discomfort or upset to anyone.

I don't want to start arguing over semantics but since there was no hard -1, there was legitimacy to land this PR. And I think we should settle down here and take some time because I worry we have some bad steam building and escalation.

I also don't want to escalate, but this is our main and formal medium for discussion. IMHO it is well established and legitimate to give critique, and suggest changes and even reversion.
On my part nothing is personal, or accusatory.

I some wish it to be private, I'd even second it but if you all deem it not necessary, fine by me.

I tend to shy away from private discussion, that is not our default mode, and IMHO should be kept only for embargoed subject or when private personal information is discussed.

All of this is codified in https://github.com/nodejs/community-committee/blob/master/GOVERNANCE.md#commcomm-meetings

@refack
Copy link
Contributor

refack commented Sep 25, 2018

I don't want to start arguing over semantics but since there was no hard -1, there was legitimacy to land this PR.

I'm also not disputing the "legitimacy" of any action done. IMHO the actions taken here fall well within the good judgement of the actors. But IMHO a mistake might have been made, and we should figure out if we need to fix it.

@MylesBorins
Copy link
Contributor

MylesBorins commented Sep 25, 2018

So here's a suggestion for how you handle this based on my experience with landing changes to charters, and to attempt to get all the changes y'all want to see land happen asap

  1. Revert the change
  1. Get consensus in this PR (or another PR) on the exact changes already approved by the board
  1. Open a new PR to make additional changes to the approved text and reach consensus on that
  1. Get the changes from the new PR brought to foundation reps ASAP to get reviewed prior to the next meeting
  2. Either via email or In the next meeting we first confirm consensus on already approved text and then get approval on the changes separately.
  3. If legal or the board have an issue with the supplementary changes we can fix those without blocking this change

edit: updated with current status

bnb added a commit that referenced this pull request Sep 25, 2018
author: @hackygolucky 

Original PR (reverted for further discussion): #380
MylesBorins pushed a commit to MylesBorins/community-committee that referenced this pull request Sep 25, 2018
author: @hackygolucky

Original PR (reverted for further discussion): nodejs#380
@keywordnew keywordnew deleted the hackygolucky-patch-9 branch October 6, 2018 23:37
@keywordnew keywordnew restored the hackygolucky-patch-9 branch October 6, 2018 23:38
@keywordnew keywordnew deleted the hackygolucky-patch-9 branch January 9, 2020 17:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants