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: Ludovic Courtès
Subject: Re: [PATCH 4/5] guix lint: Make sure a synopsis cannot start with a lower-case article.
Date: Sun, 28 Sep 2014 18:11:14 +0200
User-agent: Gnus/5.130011 (Ma Gnus v0.11) Emacs/24.3 (gnu/linux)

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?

Ludo’.



reply via email to

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