diff options
author | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2016-05-30 13:46:47 +0200 |
---|---|---|
committer | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2016-05-30 13:53:25 +0200 |
commit | a55e8f109fbaec1bb2db19a37a6537d8833c995c (patch) | |
tree | 12fbb211730776614c224dbf6f14aec11fc3a3f1 /lib/ci/gitlab_ci_yaml_processor.rb | |
parent | 43c35b0f20fb3bb67ea2b96bf8f806c7e95b6aec (diff) | |
download | gitlab-ce-a55e8f109fbaec1bb2db19a37a6537d8833c995c.tar.gz |
Enable Style/NegatedIf Rubocop coprubocop/enable-negatedif-style-cop
Favor `unless` over `if` for negative conditions
(or control flow ||).
See #17478
Diffstat (limited to 'lib/ci/gitlab_ci_yaml_processor.rb')
-rw-r--r-- | lib/ci/gitlab_ci_yaml_processor.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/ci/gitlab_ci_yaml_processor.rb b/lib/ci/gitlab_ci_yaml_processor.rb index e4b4760c53b..026a5ac97ca 100644 --- a/lib/ci/gitlab_ci_yaml_processor.rb +++ b/lib/ci/gitlab_ci_yaml_processor.rb @@ -265,7 +265,7 @@ module Ci end def validate_job_dependencies!(name, job) - if !validate_array_of_strings(job[:dependencies]) + unless validate_array_of_strings(job[:dependencies]) raise ValidationError, "#{name} job: dependencies parameter should be an array of strings" end |