diff options
| author | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2016-05-30 12:18:27 +0200 |
|---|---|---|
| committer | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2016-05-30 12:18:27 +0200 |
| commit | 7ee9b8e758cb4557afb083b78e56b39f16c0dc57 (patch) | |
| tree | a15621ab34d29aa76e8d47f56d5e59bcc890d17b | |
| parent | 43c35b0f20fb3bb67ea2b96bf8f806c7e95b6aec (diff) | |
| download | gitlab-ce-7ee9b8e758cb4557afb083b78e56b39f16c0dc57.tar.gz | |
Enable Style/MultilineIfThen rubocop copstyle/enable-multiline-if-then-rubocop-cop
Do not use then for multi-line if.
See #7478.
| -rw-r--r-- | .rubocop.yml | 2 | ||||
| -rw-r--r-- | app/models/network/graph.rb | 10 |
2 files changed, 6 insertions, 6 deletions
diff --git a/.rubocop.yml b/.rubocop.yml index 2d8eb4077f3..5cbed9a4d3d 100644 --- a/.rubocop.yml +++ b/.rubocop.yml @@ -362,7 +362,7 @@ Style/MultilineHashBraceLayout: # Do not use then for multi-line if/unless. Style/MultilineIfThen: - Enabled: false + Enabled: true # Checks that the closing brace in a method call is either on the same line as # the last method argument, or a new line. diff --git a/app/models/network/graph.rb b/app/models/network/graph.rb index 9259cb1a0fa..c71c444a669 100644 --- a/app/models/network/graph.rb +++ b/app/models/network/graph.rb @@ -89,7 +89,7 @@ module Network end end - if self.class.max_count / 2 < offset then + if self.class.max_count / 2 < offset # get max index that commit is displayed in the center. offset - self.class.max_count / 2 else @@ -130,7 +130,7 @@ module Network commit.parents(@map).each do |parent| range = commit.time..parent.time - space = if commit.space >= parent.space then + space = if commit.space >= parent.space find_free_parent_space(range, parent.space, -1, commit.space) else find_free_parent_space(range, commit.space, -1, parent.space) @@ -144,7 +144,7 @@ module Network end def find_free_parent_space(range, space_base, space_step, space_default) - if is_overlap?(range, space_default) then + if is_overlap?(range, space_default) find_free_space(range, space_step, space_base, space_default) else space_default @@ -155,7 +155,7 @@ module Network range.each do |i| if i != range.first && i != range.last && - @commits[i].spaces.include?(overlap_space) then + @commits[i].spaces.include?(overlap_space) return true; end @@ -233,7 +233,7 @@ module Network space = space_default while reserved.include?(space) do space += space_step - if space < space_base then + if space < space_base space_step *= -1 space = space_base + space_step end |