Skip to content

Commit 278c764

Browse files
add note about resolving comments as dev
1 parent 63f9c46 commit 278c764

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

source/WorkingPractices/reviews.rst

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,12 @@ Trivial tickets are an exception and do not require a SciTech review.
3737
then another person should be brought in to finish the review process and
3838
provide that external viewpoint.
3939

40+
.. tip::
41+
The reviewer will likely leave comments all of which should be addressed by
42+
the developer. The developer should **not** resolve these comments
43+
themselves, instead allowing the reviewer to resolve them when they are
44+
happy with the response.
45+
4046

4147
Preparing for Review
4248
--------------------

0 commit comments

Comments
 (0)