[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’.