Fixed linter errors [skip ci]

ankane · Aug 8, 2018 · fe76380 · fe76380
1 parent 51934a9
commit fe76380
Show file tree

Hide file tree

Showing 5 changed files with 3 additions and 6 deletions.
diff --git a/app/controllers/blazer/base_controller.rb b/app/controllers/blazer/base_controller.rb
@@ -7,7 +7,7 @@ class BaseController < ApplicationController
       skip_after_action(*filters, raise: false)
       skip_around_action(*filters, raise: false)
     else
-      skip_action_callback *filters
+      skip_action_callback(*filters)
     end
 
     protect_from_forgery with: :exception

diff --git a/app/controllers/blazer/queries_controller.rb b/app/controllers/blazer/queries_controller.rb
@@ -210,7 +210,7 @@ def render_run
         @first_row = @rows.first || []
         @column_types = []
         if @rows.any?
-          @columns.each_with_index do |column, i|
+          @columns.each_with_index do |_, i|
             @column_types << (
               case @first_row[i]
               when Integer

diff --git a/lib/blazer.rb b/lib/blazer.rb
@@ -126,8 +126,6 @@ def self.run_checks(schedule: nil)
   end
 
   def self.run_check(check)
-    rows = nil
-    error = nil
     tries = 1
 
     ActiveSupport::Notifications.instrument("run_check.blazer", check_id: check.id, query_id: check.query.id, state_was: check.state) do |instrument|

diff --git a/lib/blazer/data_source.rb b/lib/blazer/data_source.rb
@@ -90,7 +90,6 @@ def delete_results(run_id)
     end
 
     def run_statement(statement, options = {})
-      run_id = options[:run_id]
       async = options[:async]
       result = nil
       if cache_mode != "off"

diff --git a/lib/tasks/blazer.rake b/lib/tasks/blazer.rake
@@ -1,6 +1,6 @@
 namespace :blazer do
   desc "run checks"
-  task :run_checks, [:schedule] => :environment do |t, args|
+  task :run_checks, [:schedule] => :environment do |_, args|
     Blazer.run_checks(schedule: args[:schedule] || ENV["SCHEDULE"])
   end