squash: add note to onboarding doc

nodejs · Dec 30, 2017 · beaafb3 · beaafb3
1 parent 295ae9b
commit beaafb3
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/doc/onboarding.md b/doc/onboarding.md
@@ -121,6 +121,9 @@ onboarding session.
   avoid stalling the pull request.
   * Note that they are nits when you comment: `Nit: change foo() to bar().`
   * If they are stalling the pull request, fix them yourself on merge.
+* Insofar as possible, issues should be identified by tools rather than human
+  reviewers. If you are leaving comments that could be identified by tools but
+  are not, consider implementing the necessary tooling.
 * Minimum wait for comments time
   * There is a minimum waiting time which we try to respect for non-trivial
       changes, so that people who may have important input in such a distributed