lint: Check for trailing whitespace in description.

* guix/lint.scm (check-description-style): Check for trailing whitespace.
* tests/lint.scm: ("description: trailing whitespace"): New test.

Signed-off-by: Ludovic Courtès <ludo@gnu.org>
This commit is contained in:
Xinglu Chen 2021-05-30 22:30:32 +02:00 committed by Ludovic Courtès
parent 04afb76958
commit 93d85deae6
No known key found for this signature in database
GPG key ID: 090B11993D9AEBB5
2 changed files with 17 additions and 0 deletions

View file

@ -300,6 +300,15 @@ (define (check-end-of-sentence-space description)
infractions)
#:field 'description)))))
(define (check-no-trailing-whitespace description)
"Check that DESCRIPTION doesn't have trailing whitespace."
(if (string-suffix? " " description)
(list
(make-warning package
(G_ "description contains trailing whitespace")
#:field 'description))
'()))
(let ((description (package-description package)))
(if (string? description)
(append
@ -309,6 +318,7 @@ (define (check-end-of-sentence-space description)
;; Use raw description for this because Texinfo rendering
;; automatically fixes end of sentence space.
(check-end-of-sentence-space description)
(check-no-trailing-whitespace description)
(match (check-texinfo-markup description)
((and warning (? lint-warning?)) (list warning))
(plain-description

View file

@ -160,6 +160,13 @@ (define (warning-contains? str warnings)
(description "This is a 'quoted' thing."))))
(check-description-style pkg))))
(test-equal "description: trailing whitespace"
"description contains trailing whitespace"
(single-lint-warning-message
(let ((pkg (dummy-package "x"
(description "Whitespace. "))))
(check-description-style pkg))))
(test-equal "synopsis: not a string"
"invalid synopsis: #f"
(single-lint-warning-message