diff options
| -rw-r--r-- | guix/scripts/lint.scm | 9 | ||||
| -rw-r--r-- | tests/lint.scm | 10 |
2 files changed, 19 insertions, 0 deletions
diff --git a/guix/scripts/lint.scm b/guix/scripts/lint.scm index 3f0a8d4fc0..fd9fd7b931 100644 --- a/guix/scripts/lint.scm +++ b/guix/scripts/lint.scm @@ -167,12 +167,21 @@ Run a set of checkers on the specified package; if none is specified, run the ch "synopsis should start with an upper-case letter" 'synopsis))) + (define (check-start-with-package-name synopsis) + (let ((idx (string-contains-ci synopsis (package-name package)))) + (when (and idx + (= idx 0)) + (emit-warning package + "synopsis should not start with the package name") + 'synopsis))) + (let ((synopsis (package-synopsis package))) (if (string? synopsis) (begin (check-synopsis-start-upper-case synopsis) (check-final-period synopsis) (check-start-article synopsis) + (check-start-with-package-name synopsis) (check-synopsis-length synopsis))))) (define (check-patches package) diff --git a/tests/lint.scm b/tests/lint.scm index c439faa30e..62a9df90d8 100644 --- a/tests/lint.scm +++ b/tests/lint.scm @@ -124,6 +124,16 @@ (check-synopsis-style pkg)))) "synopsis should be less than 80 characters long"))) +(test-assert "synopsis: start with package name" + (->bool + (string-contains (call-with-warnings + (lambda () + (let ((pkg (dummy-package "x" + (name "foo") + (synopsis "foo, a nice package")))) + (check-synopsis-style pkg)))) + "synopsis should not start with the package name"))) + (test-assert "inputs: pkg-config is probably a native input" (->bool (string-contains |