diff options
| author | Dmitriy Zaporozhets <dzaporozhets@gitlab.com> | 2015-05-11 10:00:42 +0000 |
|---|---|---|
| committer | Dmitriy Zaporozhets <dzaporozhets@gitlab.com> | 2015-05-11 10:00:42 +0000 |
| commit | 396aba5e1032ac3f94dca85636ffa069c44e2360 (patch) | |
| tree | 4f5355b6f137fc66d988472f18012c53fd67c22c | |
| parent | 6fdc51f81d013a1e01c0742e81cdd0fa5476e2d6 (diff) | |
| parent | 0a47d8f3f418fe2e46c3bfe0f723ca0ff63e814c (diff) | |
| download | gitlab-ce-396aba5e1032ac3f94dca85636ffa069c44e2360.tar.gz | |
Merge branch 'brakeman-system' into 'master'
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 .
See merge request !1819
| -rw-r--r-- | lib/tasks/brakeman.rake | 2 |
1 files changed, 1 insertions, 1 deletions
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' |