-
Notifications
You must be signed in to change notification settings - Fork 155
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
doc: update readme for CPC #182
doc: update readme for CPC #182
Conversation
c30f569
to
b8ea5de
Compare
@MylesBorins would you be willing to also add the list of voting members to at least partially address #181? |
@kborchers I don't have the list in front of me right now and am a bit heads down for the rest of the day. Feel free to edit / push to this branch. You should have the permissions to do so |
@MylesBorins I pushed a change. Feel free to edit |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@openjs-foundation/cpc would like to get this reviewed/landed so that regular members can start PRing themselves in. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm save for my comment
I think all comments have been addressed. The sections for regular members / observers is still there but commented out until we have people to add there. Will land on wednesday if there are no objections |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few typos, but also this lists a total of 16 people in the CPC? And why keep the bootstrap README in the current branch when it's (also) safely in history?
Currently still specific to Bootstrap. Old README is archived in artifacts Refs: https://github.com/nodejs/TSC/blob/master/README.md
b5a1f3f
to
49c759b
Compare
LGTM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Currently still specific to Bootstrap.
Old README is archived in artifacts
Refs: https://github.com/nodejs/TSC/blob/master/README.md