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