guix-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 4/5] guix lint: Make sure a synopsis cannot start with a lowe


From: Cyril Roelandt
Subject: Re: [PATCH 4/5] guix lint: Make sure a synopsis cannot start with a lower-case article.
Date: Mon, 06 Oct 2014 02:28:37 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Icedove/31.0

On 09/28/2014 06:11 PM, Ludovic Courtès wrote:
> Cyril Roelandt <address@hidden> skribis:
> 
>> * guix/scripts/lint.scm (check-start-article): use "string-ci=?" instead of 
>> "string=?".
>> * tests/lint.scm: Add corresponding tests.
> 
> Test names, lines wrapped please.  :-)
> 
>>    (define (check-start-article synopsis)
>> -   (if (or (string=? (string-take synopsis 2) "A ")
>> -           (string=? (string-take synopsis 3) "An "))
>> +   (if (or (string-ci=? (string-take synopsis 2) "A ")
>> +           (string-ci=? (string-take synopsis 3) "An "))
> 
> There’s also a problem if SYNOPSIS is shorter, but I wonder if we should
> worry about it?
> 

Nah, doesn't seem so bad.

Cyril.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]