]> git.puffer.fish Git - matthieu/frr.git/commitdiff
developer: workflow.rst - add text on how to respond to reviews/comments
authorLou Berger <lberger@labn.net>
Tue, 27 Mar 2018 12:08:15 +0000 (08:08 -0400)
committerLou Berger <lberger@labn.net>
Tue, 27 Mar 2018 12:08:15 +0000 (08:08 -0400)
   add note that author's shouldn't remove a comment or review.
           also add a note that automatically generated comments can
           be deleted by authors and others when such comments are not
           the most results from that automated comment source.

Signed-off-by: Lou Berger <lberger@labn.net>
doc/developer/workflow.rst

index ff66007f8da35ddba658ddd64192be048f58f393..ac74665e8b1f32a58937544d3aa1cd4c1a68edc4 100644 (file)
@@ -256,7 +256,15 @@ After submitting your changes
 
    -  Wait for reviewers. Someone will review your code or be assigned
       to review your code.
-   -  Respond to any comments or concerns the reviewer has.
+   -  Respond to any comments or concerns the reviewer has.  Use e-mail or
+      add a comment via github to respond or to let the reviewer know how
+      their comment or concern is addressed.
+   -  An author must never delete or manually dismiss someone else's comments
+      or review.  (A review may be overridden by agreement in the weekly
+      technical meeting.)
+   -  Automatically generated comments, e.g., those generated by CI systems,
+      may be deleted by authors and others when such comments are not the most
+      results from that automated comment source.
    -  After all comments and concerns are addressed, expect your patch
       to be merged.