Enable Style/For rubocop cop

Do not use for, unless you know exactly why.

See #17478
This commit is contained in:
Grzegorz Bizon 2016-06-01 11:18:11 +02:00
parent 473ea1e974
commit 0319d5b632
2 changed files with 4 additions and 4 deletions

View File

@ -247,7 +247,7 @@ Style/FlipFlop:
# Checks use of for or each in multiline loops. # Checks use of for or each in multiline loops.
Style/For: Style/For:
Enabled: false Enabled: true
# Enforce the use of Kernel#sprintf, Kernel#format or String#%. # Enforce the use of Kernel#sprintf, Kernel#format or String#%.
Style/FormatString: Style/FormatString:

View File

@ -205,7 +205,7 @@ module Network
# Visit branching chains # Visit branching chains
leaves.each do |l| leaves.each do |l|
parents = l.parents(@map).select{|p| p.space.zero?} parents = l.parents(@map).select{|p| p.space.zero?}
for p in parents parents.each do |p|
place_chain(p, l.time) place_chain(p, l.time)
end end
end end
@ -223,7 +223,7 @@ module Network
end end
def mark_reserved(time_range, space) def mark_reserved(time_range, space)
for day in time_range time_range.each do |day|
@reserved[day].push(space) @reserved[day].push(space)
end end
end end
@ -232,7 +232,7 @@ module Network
space_default ||= space_base space_default ||= space_base
reserved = [] reserved = []
for day in time_range time_range.each do |day|
reserved.push(*@reserved[day]) reserved.push(*@reserved[day])
end end
reserved.uniq! reserved.uniq!