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

doc: update members list - #58

Merged
MylesBorins merged 1 commit into
masterfrom
update-members
Apr 5, 2018
Merged

doc: update members list#58
MylesBorins merged 1 commit into
masterfrom
update-members

Conversation

@MylesBorins

@MylesBorinsMylesBorins commented Apr 3, 2018•
edited
Loading

Copy link
Copy Markdown
Contributor
  • sync with with ncu-team sync

#31 should have added @TheLarkInn to the members list. It was closed
for unrelated reasons but we never got Sean on the list.

  • s/Modules Team Members/Members

  • remove hyperlink from "Members" header

The original hyperlink lead to:

https://github.com/orgs/nodejs/teams/modules/members

Which is only accessible to people in the Node.js org afaict. The
member list is in the doc, the link is likely to just confuse folks.

  • update ncu-team meta data

It now can inline Members and Observers. This is connected to the ACL
in github. The Observers header is currently commented out.

  • fix link in governance to membership list

@MylesBorins
MylesBorinsforce-pushed the update-members branch 2 times, most recently from 8f26e00 to 379c9aeCompareApril 3, 2018 04:10
hiroppy

This comment was marked as off-topic.

Sign in to view

devsnek

This comment was marked as off-topic.

Sign in to view

ljharb

This comment was marked as off-topic.

Sign in to view

weswigham

This comment was marked as off-topic.

Sign in to view

@MylesBorins
doc: update members list
0615965
* sync with with ncu-team sync
\#31 should have added @TheLarkInn to the members list. It was closed
for unrelated reasons but we never got Sean on the list.
* s/Modules Team Members/Members
* remove hyperlink from "Members" header
The original hyperlink lead to:
https://github.com/orgs/nodejs/teams/modules/members
Which is only accessible to people in the Node.js org afaict. The
member list is in the doc, the link is likely to just confuse folks.
* update ncu-team meta data
It now can inline Members and Observers. This is connected to the ACL
in github. The Observers header is currently commented out.
* fix link in governance to membership list
@MylesBorins

MylesBorins commented Apr 3, 2018

Copy link
Copy Markdown
ContributorAuthor

fixed a few more errata... initial post is now up to date with all changes

@MylesBorinsMylesBorins mentioned this pull request Apr 3, 2018
Merged
targos

This comment was marked as off-topic.

Sign in to view

benjamingr

This comment was marked as off-topic.

Sign in to view

mcollina

This comment was marked as off-topic.

Sign in to view

@MylesBorins
MylesBorins merged commit 1faee5e into masterApr 5, 2018
@ljharb
ljharb deleted the update-members branch April 5, 2018 17:34
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

7 more reviewers
@ljharbljharbljharb approved these changes
@mcollinamcollinamcollina approved these changes
@benjamingrbenjamingrbenjamingr approved these changes
@targostargostargos approved these changes
@weswighamweswighamweswigham approved these changes
@hiroppyhiroppyhiroppy approved these changes
@devsnekdevsnekdevsnek approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

errata

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@MylesBorins@ljharb@mcollina@benjamingr@targos@weswigham@hiroppy@devsnek