Fix codeclimate issues

lib/timber/context.rb

@@ -50,7 +50,7 @@ def json_shell(&block)
       self.class.json_shell(&block)
     end
 
-    def inspect(*args)
+    def inspect(*_args)
       "#<#{self.class.name}:#{object_id} ...>"
     end
 

lib/timber/internal_logger.rb

@@ -11,7 +11,7 @@ class Formatter < ::Logger::Formatter
       TAG = "[Timber]"
 
       # This method is invoked when a log event occurs
-      def call(severity, timestamp, progname, msg)
+      def call(_severity, _timestamp, _progname, msg)
         "#{TAG} #{String === msg ? msg : msg.inspect}\n"
       end
     end

lib/timber/log_devices/heroku_logplex.rb

@@ -3,7 +3,7 @@
 module Timber
   module LogDevices
     class HerokuLogplex < IO
-      def initialize(options = {})
+      def initialize(_options = {})
         super(STDOUT)
         if formatter.is_a?(IO::HybridFormatter)
           formatter.extend HybridFormatter

lib/timber/macros/compactor.rb

@@ -5,8 +5,8 @@ def self.compact(hash)
         new_hash = {}
         hash.each do |k, v|
           deep_v = v.is_a?(Hash) ? compact(v) : v
-          if !v.nil? && v != [] && v != {}
-            new_hash[k] = v
+          if !deep_v.nil? && deep_v != [] && deep_v != {}
+            new_hash[k] = deep_v
           end
         end
         new_hash

lib/timber/probes/active_support_log_subscriber/action_controller.rb

@@ -3,7 +3,8 @@ module Probes
     class ActiveSupportLogSubscriber < Probe
       module ActionController
         def self.process_action(_log_subscriber, event, &_block)
-          if context = CurrentContext.get(Contexts::HTTPResponses::ActionController)
+          context = CurrentContext.get(Contexts::HTTPResponses::ActionController)
+          if context
             context.event = event
           end
           yield

spec/timber/macros/compactor_spec.rb

@@ -0,0 +1,19 @@
+require "spec_helper"
+
+describe Timber::Macros::Compactor do
+  describe ".compact" do
+    let(:hash) { {} }
+    subject { described_class.compact(hash) }
+    it { should eq({}) }
+
+    context "nested" do
+      let(:hash) { {:whatever => {:nested => nil}} }
+      it { should eq({}) }
+    end
+
+    context "nested with other values" do
+      let(:hash) { {:whatever => {:nested => nil, :with_val => 1}, :another => 1} }
+      it { should eq({:whatever => {:with_val => 1}, :another => 1}) }
+    end
+  end
+end