Re: [PATCH 1/2] scripts: add guix lint

[Ludovic Courtès]

Two things I forgot to ask for (you’ll hate me for that ;-)):

  1. Could you write tests for the checkers, as tests/lint.scm?

     The tests would make sure that for a ‘package’ structure that
     exposes the problem being checked for, the checker does emit a
     warning on ‘guix-warning-port’, like:

     (define (call-with-warnings thunk)
       (let ((port (open-output-string)))
         (parameterize ((guix-warning-port port))
           (thunk))
         (get-output-string port)))

     (test-assert "synopsis warning emitted"
       (->bool
        (string-contains (call-with-warnings
                           (lambda ()
                             (check-foo-bar pkg)))
                         "synopsis has a problem")))

   2. Add an “Invoking guix lint” node in guix.texi, with links to there
      from “Defining Packages”.

TIA.  :-)

Ludo’.