Re: [PATCH v2] Enable Travis-CI as a backup continuous integration service.

Subject: Re: [PATCH v2] Enable Travis-CI as a backup continuous integration service.

Date: Fri, 16 May 2014 12:13:45 +0100

To: W. Trevor King, Wael Nasreddine

Cc: notmuch@notmuchmail.org

From: David Edmondson


On Thu, May 15 2014, W. Trevor King wrote:
> On Thu, May 15, 2014 at 10:40:06AM -0700, Wael Nasreddine wrote:
>> David Edmondson writes:
>> > I realise that you might answer "I will keep this up to date", but
>> > we have to worry about what happens if you lose interest and
>> > wander away.
>> 
>> Absolutely, I understand your point and no one can guarantee
>> maintainer-ship. I can modify my patch and add documentation
>> (comments in the yaml file) about what each flag does, where can you
>> documentation about it and of course details about the hack. Would
>> that be helpful?
>
> I personally love comments like this, but I prefer them in the commit
> message.  Otherwise the source becomes one humongous comment with a
> bit of interspersed code ;).  An interactive blame (like you get with
> tig [1] and probably any interactive Git frontent) makes these
> commit-messages easily accessible (assuming they're not buried under
> whitespace churn, etc.)

I disagree. In this case, the comments explain the code, so they belong
in the code.

When the version number of zlib we have to download changes, that's the
type of comment that belongs in the commit message.
signature.asc (application/pgp-signature)

Thread: