Skip to content

Commit beaafb3

Browse files
committed
squash: add note to onboarding doc
1 parent 295ae9b commit beaafb3

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

doc/onboarding.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -121,6 +121,9 @@ onboarding session.
121121
avoid stalling the pull request.
122122
* Note that they are nits when you comment: `Nit: change foo() to bar().`
123123
* If they are stalling the pull request, fix them yourself on merge.
124+
* Insofar as possible, issues should be identified by tools rather than human
125+
reviewers. If you are leaving comments that could be identified by tools but
126+
are not, consider implementing the necessary tooling.
124127
* Minimum wait for comments time
125128
* There is a minimum waiting time which we try to respect for non-trivial
126129
changes, so that people who may have important input in such a distributed

0 commit comments

Comments
 (0)