From 9aed3201aad270635259b57e6f9fabc0cc11f481 Mon Sep 17 00:00:00 2001 From: Jeremy Dunck Date: Sun, 7 Jul 2013 21:17:10 -0700 Subject: [PATCH] Clarified trac patch-related field meaning There was some confusion on the mentorship list as to whether 'needs tests' meant that a given ticket might not need tests, as opposed to flagging a specific patch as still needing tests. This commit seeks to clarify that confusion. --- docs/internals/contributing/triaging-tickets.txt | 3 +++ 1 file changed, 3 insertions(+) diff --git a/docs/internals/contributing/triaging-tickets.txt b/docs/internals/contributing/triaging-tickets.txt index 7bb59bc329..2aa8aaedba 100644 --- a/docs/internals/contributing/triaging-tickets.txt +++ b/docs/internals/contributing/triaging-tickets.txt @@ -175,6 +175,9 @@ This means the ticket has an associated :doc:`patch`. These will be reviewed to see if the patch is "good". +The following three fields (Needs documentation, Needs tests, +Patch needs improvement) apply only if a patch has been supplied. + Needs documentation ~~~~~~~~~~~~~~~~~~~