bug#47028: [PATCH 2/2] lint: Warn about single-character package names.

2021-04-01 Thread Tobias Geerinckx-Rice via Bug reports for GNU Guix
zimoun writes: My point is: I am not even sure that “r” should be whitelisted. I think it deserves the name, but my reasons are fuzzy and feely. Anyway: I added that exception for ‘r’ and pushed as 1126bb9cf33f10f004a5f53331389c777c025e75 et al. Kind regards, T G-R signature.asc Descript

bug#47028: [PATCH 2/2] lint: Warn about single-character package names.

2021-04-01 Thread Tobias Geerinckx-Rice via Bug reports for GNU Guix
zimoun writes: Maybe the length can be negative or zero. ;-) ‘Defensive programming’! Thanks, :-) T G-R

bug#47028: [PATCH 2/2] lint: Warn about single-character package names.

2021-03-26 Thread zimoun
Hi, On Tue, 09 Mar 2021 at 22:41, Tobias Geerinckx-Rice wrote: > +(define (check-name package) > + "Check whether PACKAGE's name matches our guidelines." > + ;; Currently checks only whether the name is too short. > + (if (<= (string-length (package-name package)) 1) Maybe the length can be

bug#47028: [PATCH 2/2] lint: Warn about single-character package names.

2021-03-09 Thread Tobias Geerinckx-Rice via Bug reports for GNU Guix
* guix/lint.scm (check-name): New procedure. (%local-checkers): Add it. --- guix/lint.scm | 16 +++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/guix/lint.scm b/guix/lint.scm index 311bc94cc3..5144fa139d 100644 --- a/guix/lint.scm +++ b/guix/lint.scm @@ -6,7 +6,7 @@