Subject: [bug#59845] [PATCH 0/4] Add PMB (Integrated Library System)

From: Ludovic Courtès <[email-masked]>
Date: Tue, 20 Dec 2022 15:56:49 +0100

Hi!

Yarl Baudig <[email-masked]> skribis:

> * gnu/packages/ils.scm: New file.
> * gnu/local.mk: Add it.

The gnu/packages changes LGTM and to qa.guix!

Minor suggestions:

> +    (synopsis "Integrated Library System")

Maybe “Integrated library system”?

> +    (description "PMB is an Integrated Library System for librarians, documentalists and

… and “@acronym{ILS, integrated library system}”.

> +information managers.
> +
> +This contains only the PMB archive (html, php and css files), you will need a web
> +server, php and a SQL database server.  See the guix pmb service.")

Please capitalize things: HTML, PHP, CSS.  You can remove “See …” at the
end.

Ludo’.