On Thu, 17 Dec 2015, RjOllos wrote:
The ticket guidelines for Trac suggest adding [patch] to the ticket summary, in
cases that the user has submitted a patch (1, 2). In some cases the submitted
patches are worthless, so there's no sense in having this keyword in the
ticket summary. In other cases the ticket author didn't include the keyword so
a ticket triager may be modifying the ticket summary.
However, we generally don't want to repeatedly change the ticket summary. One
reason is that the emails won't form in a thread in most or all email clients
when the summary has changed.
We also suggest using the "patch" keyword (3), which I think is sufficient to
label tickets that have patches. We can easily add/remove the keyword depending on the
usefulness of the patch, and ticket queries can be easily executed
to find all tickets with patches.
Any other considerations that I'm overlooking?
Yes. A major one: Adding "[patch]" is a common method and not Trac
specific. If you try to change that for Trac then you only increase
confusion.
Ciao
--
http://www.dstoecker.eu/ (PGP key available)
--
You received this message because you are subscribed to the Google Groups "Trac
Development" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to trac-dev+unsubscr...@googlegroups.com.
To post to this group, send email to trac-dev@googlegroups.com.
Visit this group at https://groups.google.com/group/trac-dev.
For more options, visit https://groups.google.com/d/optout.
