Fix cookstyle warnings

diff --git a/.rubocop.yml b/.rubocop.yml
index 649fc64dc4e1ab442d295463e9eae0b87bcff42b..044181e0d2849a845d320aacf0ae2fc9146e384e 100644 (file)
--- a/.rubocop.yml
+++ b/.rubocop.yml
@@ -15,6 +15,9 @@ Layout/HeredocIndentation:
 Naming/MethodParameterName:
   Enabled: false
 
+Style/CommandLiteral:
+  EnforcedStyle: percent_x
+
 Style/HashSyntax:
   EnforcedStyle: hash_rockets
 

diff --git a/cookbooks/networking/attributes/default.rb b/cookbooks/networking/attributes/default.rb
index d2ec5957d1a36e9db8dd093ae3de37cba096de46..6881fcafeb1f1d226779d80b4d529ee30bb9d6c2 100644 (file)
--- a/cookbooks/networking/attributes/default.rb
+++ b/cookbooks/networking/attributes/default.rb
@@ -15,6 +15,6 @@ default[:networking][:search] = []
 default[:networking][:dnssec] = "allow-downgrade"
 default[:networking][:hostname] = node.name
 default[:networking][:wireguard][:enabled] = false
-default[:networking][:wireguard][:address] = "fd43:e709:ea6d:1:#{wireguard_id[0,4]}:#{wireguard_id[4,4]}:#{wireguard_id[8,4]}:#{wireguard_id[12,4]}"
+default[:networking][:wireguard][:address] = "fd43:e709:ea6d:1:#{wireguard_id[0, 4]}:#{wireguard_id[4, 4]}:#{wireguard_id[8, 4]}:#{wireguard_id[12, 4]}"
 default[:networking][:wireguard][:keepalive] = false
 default[:networking][:wireguard][:peers] = []

diff --git a/cookbooks/networking/recipes/default.rb b/cookbooks/networking/recipes/default.rb
index d638d913217f5c1e4e7f7e3c863ed33c494c1abb..053d98b781f68019201e26da88216c9513b0f9ed 100644 (file)
--- a/cookbooks/networking/recipes/default.rb
+++ b/cookbooks/networking/recipes/default.rb
@@ -202,11 +202,11 @@ if node[:networking][:wireguard][:enabled]
     owner "root"
     group "systemd-network"
     mode "640"
-    content %x{wg genkey}
+    content %x(wg genkey)
     compile_time true
   end
 
-  node.default[:networking][:wireguard][:public_key] = %x{wg pubkey < /var/lib/systemd/wireguard/private.key}
+  node.default[:networking][:wireguard][:public_key] = %x(wg pubkey < /var/lib/systemd/wireguard/private.key)
 
   file "/var/lib/systemd/wireguard/preshared.key" do
     action :create_if_missing

diff --git a/hooks/pre-commit b/hooks/pre-commit
index cb7bc82d55348a9c92229c160e2cdf512b5c1282..60d6659326ff7908b9cf25fa8f9d1553c42523e5 100755 (executable)
--- a/hooks/pre-commit
+++ b/hooks/pre-commit
@@ -12,7 +12,7 @@ if IO.popen(["git", "ls-files", "--unmerged"]).read.empty?
   end.compact
 
   ruby_files = files.select do |file|
-    file =~ /\.rb$/ || (file !~ /\.erb$/ && `file --brief --mime-type #{file}` == "text/x-ruby\n")
+    file =~ /\.rb$/ || (file !~ /\.erb$/ && %x(file --brief --mime-type #{file}) == "text/x-ruby\n")
   end
 
   ok &&= system("bundle", "exec", "rubocop", *ruby_files) unless ruby_files.empty?

diff --git a/roles/grisu.rb b/roles/grisu.rb
index 45df5c5f4fcd1cc6de050a5ca9188b03a3d2b659..9aaa6d67b4ac642df8382678243676542e837d39 100644 (file)
--- a/roles/grisu.rb
+++ b/roles/grisu.rb
@@ -41,6 +41,6 @@ run_list(
   "role[gateway]",
   "role[web-storage]",
   "role[backup]",
-  "role[planet]",
+  "role[planet]"
   # "role[planetdump]"
 )