diff options
| author | Sirio Balmelli <sirio@b-ad.ch> | 2024-09-03 11:25:58 +0200 |
|---|---|---|
| committer | Sirio Balmelli <sirio@b-ad.ch> | 2024-09-06 14:57:35 +0200 |
| commit | c334175319949f6887dcab89afb32f1bb38e9f88 (patch) | |
| tree | 2e24d03d053fc961588b185fb09220a3e6fc6f3e /modules | |
| parent | a55b3f1ab41bb6d5025ebeebb4da5fd240b9b3b3 (diff) | |
nixos/github-runner: quote comma separators so as to pass shellcheck
Shellcheck complains:
> args=(
> ^-- SC2054 (warning): Use spaces, not commas, to separate array elements.
Quote the --labels argument to resolve.
Signed-off-by: Sirio Balmelli <sirio@b-ad.ch>
Diffstat (limited to 'modules')
| -rw-r--r-- | modules/services/github-runner/service.nix | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/modules/services/github-runner/service.nix b/modules/services/github-runner/service.nix index 75d6442..175b22d 100644 --- a/modules/services/github-runner/service.nix +++ b/modules/services/github-runner/service.nix @@ -90,7 +90,7 @@ in let configure = pkgs.writeShellApplication { name = "configure-github-runner-${name}"; - text = '' + text = /*bash*/'' export RUNNER_ROOT args=( @@ -98,7 +98,7 @@ in --disableupdate --work ${escapeShellArg workDir} --url ${escapeShellArg cfg.url} - --labels ${escapeShellArg (concatStringsSep "," cfg.extraLabels)} + --labels "${escapeShellArg (concatStringsSep "," cfg.extraLabels)}" ${optionalString (cfg.name != null ) "--name ${escapeShellArg cfg.name}"} ${optionalString cfg.replace "--replace"} ${optionalString (cfg.runnerGroup != null) "--runnergroup ${escapeShellArg cfg.runnerGroup}"} |