guix-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#70285] [PATCH 2/3] services: cuirass: Add new configuration options


From: Ludovic Courtès
Subject: [bug#70285] [PATCH 2/3] services: cuirass: Add new configuration options.
Date: Mon, 29 Apr 2024 23:33:31 +0200
User-agent: Gnus/5.13 (Gnus v5.13)

Hi Ahmad,

Ahmad Draidi <a.r.draidi@redscript.org> skribis:

> * gnu/services/cuirass.scm (<cuirass-configuration>): Add ttl and threads
> fields.
> (cuirass-shepherd-service): Handle the new options.
> * doc/guix.texi (Continuous Integration): Document them.

[...]

> +  (ttl              cuirass-configuration-ttl ;string
> +                    (default "30d"))

For consistency with <guix-publish-configuration>, I would make it an
integer (number of seconds).

If that’s fine with you, could you send an updated patch or let me know
if I should make this change on your behalf?

The rest of the patch series looks good to me!

Thanks,
Ludo’.





reply via email to

[Prev in Thread] Current Thread [Next in Thread]