]> granicus.if.org Git - neomutt/commitdiff
github: remove double checking question
authorTobias Angele <toogley@mailbox.org>
Sat, 12 May 2018 10:12:42 +0000 (12:12 +0200)
committerRichard Russon <rich@flatcap.org>
Sat, 12 May 2018 10:12:42 +0000 (11:12 +0100)
`Are there points in the code the reviewer needs to double check?` is mostly providing no benefits. When a PR author wants the reviewer to double check sth in particular, they can somewhere.

Additionally, passing tests should (obviously) also be part of the acceptance criteria.

.github/PULL_REQUEST_TEMPLATE.md

index 2d73a464834eaa3a73cf76cbb2026da03d389244..99ec28fd496d687e863dccf7c30a94c26aac7d02 100644 (file)
@@ -1,7 +1,5 @@
 * **What does this PR do?**
 
-* **Are there points in the code the reviewer needs to double check?**
-
 * **Screenshots (if relevant)**
 
 * **Does this PR meet the acceptance criteria?** (This is just a reminder for you,
@@ -11,7 +9,7 @@
      [doc/manual.xml.head](https://www.github.com/neomutt/neomutt/blob/master/doc/manual.xml.head)
      for that)
 
-   - All builds are passing
+   - All builds and tests are passing
 
    - Added [doxygen code documentation](https://neomutt.org/dev/doxygen)
      [syntax](http://www.stack.nl/~dimitri/doxygen/manual/docblocks.html)