diff options
| author | Salame William <william.salame@student.uclouvain.be> | 2017-12-28 17:31:53 +0100 |
|---|---|---|
| committer | Myles Borins <mylesborins@google.com> | 2018-02-12 19:28:30 -0500 |
| commit | 21c5d820bb77daedaccf7d100a8158e7b7e2ba77 (patch) | |
| tree | b4e871449d32394026a19135bfc5cbfcf481293a | |
| parent | e16d01fc94861e0d83f932d0e7611f90ef9a9268 (diff) | |
| download | node-new-21c5d820bb77daedaccf7d100a8158e7b7e2ba77.tar.gz | |
doc: add references to PR communication articles
Added some references to PR communication articles in
Helpful Ressources inside COLLABORATOR_GUIDE.md
PR-URL: https://github.com/nodejs/node/pull/17902
Fixes: https://github.com/nodejs/node/issues/16359
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Weijia Wang <starkwang@126.com>
| -rw-r--r-- | COLLABORATOR_GUIDE.md | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/COLLABORATOR_GUIDE.md b/COLLABORATOR_GUIDE.md index 9b8140bfea..dd38364eed 100644 --- a/COLLABORATOR_GUIDE.md +++ b/COLLABORATOR_GUIDE.md @@ -116,6 +116,11 @@ oppose the PR, it can be landed. Where there is disagreement among TSC members or objections from one or more Collaborators, `semver-major` pull requests should be put on the TSC meeting agenda. +#### Helpful resources + +* How to respectfully and usefully review code, part [one](https://mtlynch.io/human-code-reviews-1/) and [two](https://mtlynch.io/human-code-reviews-2/) +* [How to write a positive code review](https://css-tricks.com/code-review-etiquette/) + ### Waiting for Approvals Before landing pull requests, sufficient time should be left for input |