#2427: trac-pre-commit-hook and trac-post-commit-hook should be more lenient
about
keywords.
---------------------------------+------------------------------------------
Reporter: [EMAIL PROTECTED] | Owner: cboos
Type: enhancement | Status: new
Priority: low | Milestone: 0.11
Component: version control | Version: 0.9
Severity: minor | Resolution:
Keywords: post-commit-hook |
---------------------------------+------------------------------------------
Changes (by cboos):
* component: general => version control
* keywords: => post-commit-hook
* milestone: 0.10 => 0.11
* owner: jonas => cboos
* priority: normal => low
* severity: normal => minor
Comment:
What you're trying to achieve here is #508 (Related checkins).
However, that would be a rather brute force way to do it.
TracCrossReferences provides that functionality, and doesn't
clutter the ticket history with those references to related
checkins, rather they are displayed on a specific page reachable
by a __Cross-References__ page navigation link (available on every
page, btw).
Though the development of TracCrossReferences has stalled a bit,
it ''will'' be updated in the upcoming weeks, and I don't want
to see a widespread use of workarounds in the meantime...
In the same thinking, I don't think that the majority of people
using the trac-post-commit-hook would like to have ''every''
reference to their tickets added as comment. I'd expect them
willing to have a way to keep control (we can even be ''stricter''
on what we reference, using `--require-envelope`, on trunk).
This makes me think that your patch would eventually be
acceptable if it introduced a `--lenient` mode, playing well
with the rest (please do it for trunk, not for 0.9-stable).
I move the milestone to 0.11, but it may still make it for 0.10
if the patch is good and here in duly time (no pressure ;) )
--
Ticket URL: <http://projects.edgewall.com/trac/ticket/2427>
The Trac Project <http://trac.edgewall.com/>
_______________________________________________
Trac-Tickets mailing list
[email protected]
http://lists.edgewall.com/mailman/listinfo/trac-tickets
