summaryrefslogtreecommitdiffstats
path: root/Help/dev
diff options
context:
space:
mode:
authorBrad King <brad.king@kitware.com>2017-04-06 13:53:19 (GMT)
committerBrad King <brad.king@kitware.com>2017-04-06 13:53:19 (GMT)
commit43381b4de7dcc7ac35532141a75f50951cc0603f (patch)
treefe8f13f65499239f3cee8eed6acc94b513928d28 /Help/dev
parentda9fe11b98ae9b176c1fbee8afb2cacddacf0c11 (diff)
downloadCMake-43381b4de7dcc7ac35532141a75f50951cc0603f.zip
CMake-43381b4de7dcc7ac35532141a75f50951cc0603f.tar.gz
CMake-43381b4de7dcc7ac35532141a75f50951cc0603f.tar.bz2
Help/dev: Document preferred page for MR comments
Diffstat (limited to 'Help/dev')
-rw-r--r--Help/dev/review.rst4
1 files changed, 4 insertions, 0 deletions
diff --git a/Help/dev/review.rst b/Help/dev/review.rst
index 3004a14..3db1f40 100644
--- a/Help/dev/review.rst
+++ b/Help/dev/review.rst
@@ -106,6 +106,10 @@ Human Review
Anyone is welcome to review merge requests and make comments!
+Please make comments directly on the MR page Discussion and Changes tabs
+and not on individual commits. Comments on a commit may disappear
+from the MR page if the commit is rewritten in response.
+
Reviewers may add comments providing feedback or to acknowledge their
approval. Lines of specific forms will be extracted during the `merge`_
step and included as trailing lines of the generated merge commit message.