Salut !
Mathieu Lirzin <m...@gnu.org> skribis:
> * gnu/services/cuirass.scm: New file.
> * gnu/local.mk (GNU_SYSTEM_MODULES): Add it.
> * doc/guix.texi (Continuous integration): New node.
[...]
> +++ b/doc/guix.texi
> @@ -7687,6 +7687,7 @@ declaration.
> * Mail Services:: IMAP, POP3, SMTP, and all that.
> * Web Services:: Web servers.
> * Network File System:: NFS related services.
> +* Continuous integration:: The cuirass service.
^ ^
Capitalize please. :-)
> +@deftp {Data Type} cuirass-configuration
> +Data type representing the configuration of Cuirass.
> +
> +@table @asis
> +@item @code{cache-directory} (default: "")
^^
Could you enclose it in @code? Same for the other default values.
> +@defvar %default-cuirass-configuration
So far we’ve used “@defvr {Scheme Variable} foo”, like Guile does. It
may be debatable, but I think we should be consistent.
I agree with Leo’s suggestion about the doc, but apart from that, it
looks all good. Nice work!
Thank you! I can’t wait to see it in action on the new machine. :-)
Ludo’.
