doc: moved links about PR communication to COLLABORATOR_GUIDE.md

Moved a few links about PR communication from CONTRIBUTING.md to COLLABORATOR_GUIDE.md
nodejs · Dec 29, 2017 · 4ced007 · 4ced007
1 parent 670ddfd
commit 4ced007
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git 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

diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
@@ -828,8 +828,6 @@ The following additional resources may be of assistance:
 * [How to create a Minimal, Complete, and Verifiable example](https://stackoverflow.com/help/mcve)
 * [core-validate-commit](https://github.com/evanlucas/core-validate-commit) -
   A utility that ensures commits follow the commit formatting guidelines.
-* 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/)
 
 <a id="developers-certificate-of-origin"></a>
 ## Developer's Certificate of Origin 1.1