aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChristopher Baines <mail@cbaines.net>2018-05-28 20:31:11 +0100
committerChristopher Baines <mail@cbaines.net>2018-05-28 20:31:11 +0100
commit44751b2118a191d96c11cf5d5e5fed7915122551 (patch)
tree0982e17ddd411661e0c8fe4293679507e91f0532
parenta525b6cdeece13b5dddd286c2330ec0baae59361 (diff)
downloadgovuk-mini-environment-admin-44751b2118a191d96c11cf5d5e5fed7915122551.tar
govuk-mini-environment-admin-44751b2118a191d96c11cf5d5e5fed7915122551.tar.gz
Avoid unnecessary quoting
This seems to cause errors in the govuk-guix when parsing the arguments.
-rw-r--r--app/jobs/govuk_guix/build_job.rb2
-rw-r--r--app/jobs/govuk_guix/job.rb2
2 files changed, 1 insertions, 3 deletions
diff --git a/app/jobs/govuk_guix/build_job.rb b/app/jobs/govuk_guix/build_job.rb
index fb5e256..c4da654 100644
--- a/app/jobs/govuk_guix/build_job.rb
+++ b/app/jobs/govuk_guix/build_job.rb
@@ -87,7 +87,7 @@ class GovukGuix::BuildJob < GovukGuix::Job
def signon_user_arguments(signon_users)
signon_users.map do |signon_user|
- "--signon-user='#{signon_user_to_sexp(signon_user)}'"
+ "--signon-user=#{signon_user_to_sexp(signon_user)}"
end
end
diff --git a/app/jobs/govuk_guix/job.rb b/app/jobs/govuk_guix/job.rb
index b79d380..2a028d6 100644
--- a/app/jobs/govuk_guix/job.rb
+++ b/app/jobs/govuk_guix/job.rb
@@ -50,8 +50,6 @@ class GovukGuix::Job < Que::Job
if value == true
"--#{transformed_key}"
- elsif value.include? ' '
- "--#{transformed_key}='#{value}'"
else
"--#{transformed_key}=#{value}"
end