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

Required updates to CommComm Charter to reflect OpenJS Foundation #526

Merged
merged 9 commits into from
Nov 26, 2019

Conversation

bnb
Copy link
Contributor

@bnb bnb commented Sep 5, 2019

Here's a PR that begins to address the required Charter changes. Most of this is a strawperson, more than happy to get feedback and tweak my changes as necessary.

Further context:

  • Section 3:
    • I think this section is super important but is not able to be easily reconciled with the state of the OpenJS Foundation and CommComm's representation in the Foundation. I am not personally sure how to approach this section while retaining the intent, meaningfulness, and spirit it currently holds.
    • This needs to be resolved and is currently blocking this PR being presented to the CPC and/or Board.
  • This PR removes all references to Individual Membership Director and Director roles, since they are invalid in the existing organizational structure.
    • CommComm membership will need to be updated to reflect this.
  • Additional notes:
    • Fixed inconsistent whitespace
    • Removed references to Individual Membership Directors, since that is a role and membership class that does not currently exist in the OpenJS Foundation and the current plans for such a membership class would fall outside the scope of the Community Committee.

Updates:

  • Removed Section 3
  • Added CPC representation
    • Deduplicated some content

@bnb bnb requested a review from a team September 10, 2019 11:41
@bnb
Copy link
Contributor Author

bnb commented Sep 10, 2019

@nodejs/community-committee if y'all could help review this, that would be stellar.

@mhdawson
Copy link
Member

For Section 3, in the TSC charter update it was simply removed. I think we can just do the same here as there were no objections when the TSC charter was updated. I do think that role is important but in the current organization, it does not apply to a specific project. Instead, similar language may be applicable to the CPC.

@bnb
Copy link
Contributor Author

bnb commented Sep 19, 2019

@nodejs/community-committee please take a look – updated two things that we talked about in the meeting today ❤️

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

@bnb
Copy link
Contributor Author

bnb commented Sep 26, 2019

@nodejs/community-committee ping on this again, please review. This is an important change for us, and your feedback/approval would be greatly appreciated ❤️

Copy link
Member

@ahmadawais ahmadawais left a comment

Choose a reason for hiding this comment

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

Small suggestions. Everything else Looks good to me! 🔥

Community-Committee-Charter.md Outdated Show resolved Hide resolved
Co-Authored-By: Ahmad Awais ⚡️ <me@AhmadAwais.com>
Community-Committee-Charter.md Outdated Show resolved Hide resolved
Copy link
Member

@joesepi joesepi left a comment

Choose a reason for hiding this comment

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

Couple small nits but overall LGTM. Thanks to @bnb and other for pushing on this. Much appreciated 🙌

Community-Committee-Charter.md Outdated Show resolved Hide resolved
Community-Committee-Charter.md Outdated Show resolved Hide resolved
Copy link
Contributor

@dshaw dshaw left a comment

Choose a reason for hiding this comment

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

Thanks, @bnb. Looks solid.

Community-Committee-Charter.md Outdated Show resolved Hide resolved
Community-Committee-Charter.md Outdated Show resolved Hide resolved
@bnb
Copy link
Contributor Author

bnb commented Nov 26, 2019

This has gone through all the necessary review at the CPC level and in today's meeting it was asserted that it was approved. Thanks y'all 🎊

@bnb bnb merged commit d3ebba3 into master Nov 26, 2019
@keywordnew keywordnew deleted the bnb/charter-updates-openjs-foundation branch August 6, 2020 18:00
Copy link

@Azasyaid404 Azasyaid404 left a comment

Choose a reason for hiding this comment

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

bnb/charter-updates-openjs-foundation

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