Skip to content

doc: add text around collaborative expectations - #46121

Closed
mhdawson wants to merge 6 commits into
nodejs:mainfrom
mhdawson:add-project-health
Closed

doc: add text around collaborative expectations#46121
mhdawson wants to merge 6 commits into
nodejs:mainfrom
mhdawson:add-project-health

Conversation

@mhdawson

@mhdawsonmhdawson commented Jan 6, 2023

Copy link
Copy Markdown
Member
  • add text discussed by the TSC

Signed-off-by: Michael Dawson mdawson@devrus.com

@mhdawson
doc: add text around collaborative expectations
d008b7d
- add text discussed by the TSC
Signed-off-by: Michael Dawson <mdawson@devrus.com>
@nodejs-github-botnodejs-github-bot added the doc Issues and PRs related to the documentations. label Jan 6, 2023
mscdex
mscdex reviewed Jan 6, 2023
View reviewed changes
Comment threadREADME.md Outdated
Trott
Trott reviewed Jan 6, 2023
View reviewed changes
Comment threadREADME.md Outdated
Trott
Trott reviewed Jan 6, 2023
View reviewed changes
Comment threadREADME.md Outdated
@bnoordhuis

bnoordhuis commented Jan 7, 2023

Copy link
Copy Markdown
Member

Did something prompt this? The blurb has a strong "water is wet, news at 11" vibe and the README is already interminably long...

Aside: I'd be in favor of moving out the collaborators/triagers/etc. lists.

RaisinTen
RaisinTen reviewed Jan 9, 2023
View reviewed changes
Comment threadREADME.md Outdated
@mhdawson@Trott
incorporate suggestions
51e0afd
Co-authored-by: Rich Trott <rtrott@gmail.com>
aduh95
aduh95 reviewed Jan 9, 2023
View reviewed changes
Comment threadREADME.md Outdated
aduh95
aduh95 reviewed Jan 9, 2023
View reviewed changes
Comment threadREADME.md Outdated
mhdawsonand others added 4 commits January 9, 2023 12:30
@mhdawson@aduh95
Update README.md
66a777f
Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com>
@mhdawson@aduh95
Update README.md
4202118
Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com>
@mhdawson@Trott
incorporate suggestions
cc7fc16
Co-authored-by: Rich Trott <rtrott@gmail.com>
@mhdawson
incorporate suggested changes
febcd9e
@mhdawson

mhdawson commented Jan 9, 2023

Copy link
Copy Markdown
MemberAuthor

@Trott I've incorporate all of your suggested changes.

ChALkeR
ChALkeR approved these changes Jan 11, 2023
View reviewed changes

@ChALkeRChALkeR left a comment •
edited
Loading

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

(Would expect it below code of conduct, but that's not very signiciant.)

@mhdawson

mhdawson commented Jan 11, 2023

Copy link
Copy Markdown
MemberAuthor

I think the open questions have been discussed/answered. I meant to tag the TSC at begining with @nodejs/tsc which I'm doing now so that we get more TSC review before it lands. @nodejs/tsc.

mcollina
mcollina approved these changes Jan 11, 2023
View reviewed changes

@mcollinamcollina left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@mhdawson

mhdawson commented Jan 17, 2023

Copy link
Copy Markdown
MemberAuthor

Ok going to go ahead and land.

@mhdawson

mhdawson commented Jan 17, 2023

Copy link
Copy Markdown
MemberAuthor

Landed in c62a860

mhdawson added a commit that referenced this pull request Jan 17, 2023
@mhdawson
doc: add text around collaborative expectations
c62a860
- add text discussed by the TSC
Signed-off-by: Michael Dawson <mdawson@devrus.com>
PR-URL: #46121
Reviewed-By: Robert Nagy <ronagy@icloud.com>
Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com>
Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Beth Griggs <bethanyngriggs@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Joe Sepi <sepi@joesepi.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
RafaelGSS pushed a commit to RafaelGSS/node that referenced this pull request Jan 17, 2023
@mhdawson@RafaelGSS
doc: add text around collaborative expectations
5e01bb9
- add text discussed by the TSC
Signed-off-by: Michael Dawson <mdawson@devrus.com>
PR-URL: nodejs#46121
Reviewed-By: Robert Nagy <ronagy@icloud.com>
Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com>
Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Beth Griggs <bethanyngriggs@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Joe Sepi <sepi@joesepi.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
RafaelGSS pushed a commit that referenced this pull request Jan 20, 2023
@mhdawson@RafaelGSS
doc: add text around collaborative expectations
90924ce
- add text discussed by the TSC
Signed-off-by: Michael Dawson <mdawson@devrus.com>
PR-URL: #46121
Reviewed-By: Robert Nagy <ronagy@icloud.com>
Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com>
Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Beth Griggs <bethanyngriggs@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Joe Sepi <sepi@joesepi.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
@RafaelGSSRafaelGSS mentioned this pull request Jan 20, 2023
Merged
juanarbol pushed a commit that referenced this pull request Jan 26, 2023
@mhdawson@juanarbol
doc: add text around collaborative expectations
af04a4b
- add text discussed by the TSC
Signed-off-by: Michael Dawson <mdawson@devrus.com>
PR-URL: #46121
Reviewed-By: Robert Nagy <ronagy@icloud.com>
Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com>
Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Beth Griggs <bethanyngriggs@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Joe Sepi <sepi@joesepi.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
@juanarboljuanarbol mentioned this pull request Jan 28, 2023
Merged
juanarbol pushed a commit that referenced this pull request Jan 31, 2023
@mhdawson@juanarbol
doc: add text around collaborative expectations
5294371
- add text discussed by the TSC
Signed-off-by: Michael Dawson <mdawson@devrus.com>
PR-URL: #46121
Reviewed-By: Robert Nagy <ronagy@icloud.com>
Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com>
Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Beth Griggs <bethanyngriggs@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Joe Sepi <sepi@joesepi.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@GeoffreyBoothGeoffreyBoothGeoffreyBooth approved these changes
@TrottTrottTrott left review comments
@ronagronagronag approved these changes
@ChALkeRChALkeRChALkeR approved these changes
@mcollinamcollinamcollina approved these changes
@gireeshpunathilgireeshpunathilgireeshpunathil approved these changes
@BethGriggsBethGriggsBethGriggs approved these changes
@legendecaslegendecaslegendecas approved these changes
@aduh95aduh95aduh95 approved these changes
@RafaelGSSRafaelGSSRafaelGSS approved these changes
@RaisinTenRaisinTenRaisinTen approved these changes
+2 more reviewers
@mscdexmscdexmscdex left review comments
@joesepijoesepijoesepi approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

docIssues and PRs related to the documentations.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

16 participants

@mhdawson@bnoordhuis@mcollina@mscdex@joesepi@ChALkeR@GeoffreyBooth@Trott@ronag@gireeshpunathil@BethGriggs@legendecas@aduh95@RafaelGSS@RaisinTen@nodejs-github-bot