From 0a47d8f3f418fe2e46c3bfe0f723ca0ff63e814c Mon Sep 17 00:00:00 2001
From: Jacob Vosmaer <contact@jacobvosmaer.nl>
Date: Mon, 11 May 2015 11:11:03 +0200
Subject: [PATCH] Follow shell command guidelines in Brakeman task
There is no real issue here because the shell command is completely
static, but it is still nicer to just follow the guidelines in
doc/development/shell_commands.md .
---
lib/tasks/brakeman.rake | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/tasks/brakeman.rake b/lib/tasks/brakeman.rake
index 3a225801ff2..52a9b017e79 100644
--- a/lib/tasks/brakeman.rake
+++ b/lib/tasks/brakeman.rake
@@ -1,6 +1,6 @@
desc 'Security check via brakeman'
task :brakeman do
- if system("brakeman --skip-files lib/backup/repository.rb -w3 -z")
+ if system(*%W(brakeman --skip-files lib/backup/repository.rb -w3 -z))
puts 'Security check succeed'
else
puts 'Security check failed'
--
GitLab