[PATCH] D71916: High-Level Code-Review Documentation Update

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Feb 29 13:07:00 PST 2020


hfinkel marked 3 inline comments as done.
hfinkel added inline comments.


================
Comment at: llvm/docs/CodeReview.rst:230
+  smaller your patch is, the higher the probability that somebody will take a quick
+  look at it. When doing this, it is helpful to add "[N/M]" (for 1 < N < M) to
+  the title of each patch in the series, so it is clear that there is an order
----------------
hubert.reinterpretcast wrote:
> Is this meant to imply that the first and the final patches in a series should use a different convention?
No. Thanks for catching that.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71916/new/

https://reviews.llvm.org/D71916





More information about the llvm-commits mailing list