Previous commit ticket ref syntax restored
Simon Marlow
marlowsd at gmail.com
Thu Aug 15 15:38:20 CEST 2013
On 15/08/13 14:36, Herbert Valerio Riedel wrote:
> Hello Simon,
>
> On 2013-08-15 at 04:26:38 +0200, Simon Peyton-Jones wrote:
>> My gut feel is that it's fragile and unexpected to have tickets change
>> status based on commit messages. Revisiting the ticket itself can
>> remind you
>> that the fix is only partial, or
>> that there is another related ticket to look at, or
>> that you need to add a regression test, or
>> something else
>>
>> Too much automation can be confusing. I'd suggest (fairly strongly --
>> but resist if you feel otherwise) making status changes only based on
>> explicit user actions. It doesn't take long to do!
>
> Well, I'm somewhat convinced (and I don't have a solution to offer
> addressing your concerns at the moment).
>
> Moreover, as it's easier right now to just configure more or less what the previous
> semantics were and post-pone devising a more elaborate ticket/commit action
> command system, I've reconfigured the Trac instance as follows:
>
> - As soon as a ticket-reference matching the regexp '#[0-9]+' occurs in
> a commit's message (which became newly reachable from the `master`
> branch with the current `git push` operation), the commit is added as
> a comment to the referenced ticket.
>
> - Aside from adding comments to a ticket, no other actions are
> triggered by Git commit messages.
>
> Is this acceptable?
That's great. I also quite liked the previous behaviour.
Cheers,
Simon
More information about the ghc-devs
mailing list