]> git.openstreetmap.org Git - rails.git/commitdiff
Fix rubocop Style/CombinableLoops warnings
authorTom Hughes <tom@compton.nu>
Wed, 2 Sep 2020 17:27:11 +0000 (18:27 +0100)
committerTom Hughes <tom@compton.nu>
Wed, 2 Sep 2020 17:27:11 +0000 (18:27 +0100)
.rubocop_todo.yml
db/migrate/021_move_to_innodb.rb

index 87bf43bc3b07c9b10f37aad04189693b807a66e3..2d638e1e144c35e27393fdf5e30873c1da1c5308 100644 (file)
@@ -188,11 +188,6 @@ Rails/OutputSafety:
 Rails/TimeZone:
   Enabled: false
 
-# Offense count: 1
-Style/CombinableLoops:
-  Exclude:
-    - 'db/migrate/021_move_to_innodb.rb'
-
 # Offense count: 572
 # Cop supports --auto-correct.
 # Configuration parameters: EnforcedStyle.
index 47ebecc835aa16d6ef0ed088d075dffbe0584fed..81663eb1b24d6a1e47f680ca3279ea826f79e6d6 100644 (file)
@@ -11,11 +11,8 @@ class MoveToInnodb < ActiveRecord::Migration[4.2]
 
     @ver_tbl.each do |tbl|
       change_column tbl, "version", :bigint, :null => false
-    end
-
-    @ver_tbl.each do |tbl|
       add_column "current_#{tbl}", "version", :bigint, :null => false
-      # As the initial version of all nodes, ways and relations is 0, we set the
+      # As the initial version of all nodes, ways and relations is 0, we set tehe
       # current version to something less so that we can update the version in
       # batches of 10000
       tbl.classify.constantize.update_all(:version => -1)