diff options
author | Robert Speicher <robert@gitlab.com> | 2016-05-30 17:59:10 +0000 |
---|---|---|
committer | Robert Speicher <robert@gitlab.com> | 2016-05-30 17:59:10 +0000 |
commit | e7586cfbdc6a77b5771bea38a9ee3a9c17cdd37a (patch) | |
tree | 8c18de2755646c8e60c4554cf66d6475898b690f /app/models/ci | |
parent | ea329376302bac3bd49244ad043fa8adeb1d002e (diff) | |
parent | a55e8f109fbaec1bb2db19a37a6537d8833c995c (diff) | |
download | gitlab-ce-e7586cfbdc6a77b5771bea38a9ee3a9c17cdd37a.tar.gz |
Merge branch 'rubocop/enable-negatedif-style-cop' into 'master'
Enable Style/NegatedIf Rubocop cop
Favor `unless` over `if` for negative conditions (or control flow ||).
```ruby
# bad
do_something if !some_condition
# bad
do_something if not some_condition
# good
do_something unless some_condition
# good
some_condition || do_something
```
See #17478
See merge request !4355
Diffstat (limited to 'app/models/ci')
-rw-r--r-- | app/models/ci/runner.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/app/models/ci/runner.rb b/app/models/ci/runner.rb index 6829dc91cb9..adb65292208 100644 --- a/app/models/ci/runner.rb +++ b/app/models/ci/runner.rb @@ -60,7 +60,7 @@ module Ci end def display_name - return short_sha unless !description.blank? + return short_sha if description.blank? description end |