Skip to content

Conversation

@Trott
Copy link
Member

In a continued effort to make our GOVERNANCE.md easier to read and understand:

Revise the introductory paragraphs for Collaborator Nominations in
GOVERNANCE.md. This change splits long and unnecessarily complex
sentences into multiple simple statements.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • documentation is changed or added
  • commit message follows commit guidelines

Revise the introductory paragraphs for Collaborator Nominations in GOVERNANCE.md. This change splits long and unnecessarily complex sentences into multiple simple statements.
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-botnodejs-github-bot added the meta Issues and PRs related to the general management of the project. label Apr 15, 2019
GOVERNANCE.md Outdated
Any existing Collaborator can nominate an individual making significant
and valuable contributions across the Node.js organization to become a new
Collaborator.
Existing Collaborators can nominate someone to become a new Collaborator.
Copy link
Contributor

Choose a reason for hiding this comment

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

Is “new” adding anything here? (I know it was there before too.)

@TrottTrott added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Apr 15, 2019
Copy link
Member

@mhdawsonmhdawson left a comment

Choose a reason for hiding this comment

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

LGTM

@Trott
Copy link
MemberAuthor

Landed in ba74e42

@TrottTrott closed this Apr 17, 2019
Trott added a commit to Trott/io.js that referenced this pull request Apr 17, 2019
Revise the introductory paragraphs for Collaborator Nominations in GOVERNANCE.md. This change splits long and unnecessarily complex sentences into multiple simple statements. PR-URL: nodejs#27237 Reviewed-By: Anatoli Papirovski <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Michael Dawson <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
@TrottTrott deleted the collab-nom branch January 13, 2022 22:51
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.metaIssues and PRs related to the general management of the project.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

@Trott@nodejs-github-bot@apapirovski@jasnell@cjihrig@BridgeAR@mhdawson@mihalskiy