Zfilipin has uploaded a new change for review.
https://gerrit.wikimedia.org/r/252667
Change subject: Fixed Style/SpaceAroundOperators RuboCop offense
......................................................................
Fixed Style/SpaceAroundOperators RuboCop offense
Bug: T106220
Change-Id: I18e0c0aca7f084922af58b8a1b44ed39ed053224
---
M .rubocop_todo.yml
M lib/labs-vagrant.rb
M lib/mediawiki-vagrant/roles/list.rb
M lib/mediawiki-vagrant/roles/root.rb
4 files changed, 3 insertions(+), 12 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/vagrant
refs/changes/67/252667/1
diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml
index 272a9bc..a99d453 100644
--- a/.rubocop_todo.yml
+++ b/.rubocop_todo.yml
@@ -83,15 +83,6 @@
Exclude:
- 'lib/mediawiki-vagrant/environment.rb'
-# Offense count: 3
-# Cop supports --auto-correct.
-# Configuration parameters: MultiSpaceAllowedForOperators.
-Style/SpaceAroundOperators:
- Exclude:
- - 'lib/labs-vagrant.rb'
- - 'lib/mediawiki-vagrant/roles/list.rb'
- - 'lib/mediawiki-vagrant/roles/root.rb'
-
# Offense count: 2
# Cop supports --auto-correct.
# Configuration parameters: EnforcedStyle, EnforcedStyleForEmptyBraces,
SupportedStyles.
diff --git a/lib/labs-vagrant.rb b/lib/labs-vagrant.rb
index cb98ce1..5cddbb0 100755
--- a/lib/labs-vagrant.rb
+++ b/lib/labs-vagrant.rb
@@ -17,7 +17,7 @@
prefix = enabled.include?(role) ? '*' : ' '
"#{prefix} #{role}"
end
- col, *cols = roles.each_slice((roles.size/3.0).ceil).to_a
+ col, *cols = roles.each_slice((roles.size / 3.0).ceil).to_a
col.zip(*cols) do |a, b, c|
puts sprintf('%-26s %-26s %-26s', a, b, c)
end
diff --git a/lib/mediawiki-vagrant/roles/list.rb
b/lib/mediawiki-vagrant/roles/list.rb
index 2f5477b..fb0ea68 100644
--- a/lib/mediawiki-vagrant/roles/list.rb
+++ b/lib/mediawiki-vagrant/roles/list.rb
@@ -88,7 +88,7 @@
def print_cols(roles)
if roles.any?
- col, *cols = roles.each_slice((roles.size/3.0).ceil).to_a
+ col, *cols = roles.each_slice((roles.size / 3.0).ceil).to_a
col.zip(*cols) do |a, b, c|
@env.ui.info sprintf('%-26s %-26s %-26s', a, b, c)
end
diff --git a/lib/mediawiki-vagrant/roles/root.rb
b/lib/mediawiki-vagrant/roles/root.rb
index 5e36b58..1d3a899 100644
--- a/lib/mediawiki-vagrant/roles/root.rb
+++ b/lib/mediawiki-vagrant/roles/root.rb
@@ -60,7 +60,7 @@
end
commands.keys.sort.each do |key|
- o.separator " #{key.ljust(longest+1)} #{commands[key]}"
+ o.separator " #{key.ljust(longest + 1)} #{commands[key]}"
end
o.separator ''
--
To view, visit https://gerrit.wikimedia.org/r/252667
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I18e0c0aca7f084922af58b8a1b44ed39ed053224
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/vagrant
Gerrit-Branch: master
Gerrit-Owner: Zfilipin <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
