Merge branch 'master' of github.com:piotrmurach/github

github_api.gemspec

@@ -19,7 +19,7 @@ Gem::Specification.new do |gem|
 
   gem.add_dependency 'addressable', '~> 2.4'
   gem.add_dependency 'hashie',      '>= 3.4'
-  gem.add_dependency 'faraday',     '~> 0.8', '< 0.10'
+  gem.add_dependency 'faraday',     '~> 0.8'
   gem.add_dependency 'oauth2',      '~> 1.0'
   gem.add_dependency 'descendants_tracker', '~> 0.0.4'
   gem.add_dependency 'mime-types',  '>= 1.16', '< 3.0'

lib/github_api/api.rb

@@ -24,9 +24,9 @@ class API
     include Request::Verbs
     include RateLimit
 
-    attr_reader *Github.configuration.property_names
+    attr_reader(*Github.configuration.property_names)
 
-    attr_accessor *Validations::VALID_API_KEYS
+    attr_accessor(*Validations::VALID_API_KEYS)
 
     attr_accessor :current_options
 
@@ -209,7 +209,7 @@ def self.method_added(method_name)
     #
     # @api private
     def filter_callbacks(kind, action_name)
-      matched_callbacks = self.class.send("#{kind}_callbacks").select do |callback|
+      self.class.send("#{kind}_callbacks").select do |callback|
         callback[:only].nil? || callback[:only].include?(action_name)
       end
     end

lib/github_api/client/authorizations/app.rb

@@ -47,7 +47,7 @@ def check(*args)
       if arguments.client_id
         begin
           get_request("/applications/#{arguments.client_id}/tokens/#{arguments.access_token}", params)
-        rescue Github::Error::NotFound => e
+        rescue Github::Error::NotFound
           nil
         end
       else

lib/github_api/client/markdown.rb

@@ -49,7 +49,7 @@ def render(*args)
     #
     def render_raw(*args)
       params = arguments(args).params
-      mime_type, params['data'] = params['mime'], args.shift
+      params['data'] = args.shift
       params['raw'] = true
       params['accept'] = params.fetch('accept') { 'text/plain' }
 

lib/github_api/client/repos.rb

@@ -282,7 +282,6 @@ def contributors(*args)
       arguments(args, required: [:user, :repo]) do
         permit %w[ anon ]
       end
-      params = arguments.params
 
       response = get_request("/repos/#{arguments.user}/#{arguments.repo}/contributors", arguments.params)
       return response unless block_given?
@@ -342,7 +341,6 @@ def edit(*args)
     # @api public
     def delete(*args)
       arguments(args, required: [:user, :repo])
-      params = arguments.params
 
       delete_request("/repos/#{arguments.user}/#{arguments.repo}", arguments.params)
     end

lib/github_api/client/repos/collaborators.rb

@@ -21,7 +21,6 @@ class Client::Repos::Collaborators < API
     # @api public
     def list(*args)
       arguments(args, required: [:user, :repo])
-      params = arguments.params
 
       response = get_request("/repos/#{arguments.user}/#{arguments.repo}/collaborators", arguments.params)
       return response unless block_given?

lib/github_api/client/repos/releases/assets.rb

@@ -35,7 +35,7 @@ def list(*args)
     #
     # @api public
     def get(*args)
-      params = arguments(args, required: [:owner, :repo, :id]).params
+      arguments(args, required: [:owner, :repo, :id]).params
 
       get_request("/repos/#{arguments.owner}/#{arguments.repo}/releases/assets/#{arguments.id}" , arguments.params)
     end

lib/github_api/error.rb

@@ -21,7 +21,7 @@ def initialize(message = $!)
       end
 
       def backtrace
-        if @response_message && @response_message.respond_to?(:backtrace)
+        if @response_message.respond_to?(:backtrace)
           @response_message.backtrace
         else
           super

lib/github_api/ext/faraday.rb

@@ -7,7 +7,7 @@ class ParamsHash
       def params_encoder(encoder = nil)
         if encoder
           @encoder = encoder
-        else
+        elsif defined?(@encoder)
           @encoder
         end
       end

lib/github_api/request/jsonize.rb

@@ -32,7 +32,6 @@ def encode_body(value)
     end
 
     def request_with_body?(env)
-      type = request_type(env)
       has_body?(env) and safe_to_modify?(env)
     end
 

lib/github_api/response_wrapper.rb

@@ -19,6 +19,7 @@ def initialize(response, current_api)
       @response    = response
       @current_api = current_api
       @env         = response.env
+      @body        = nil
     end
 
     # Overwrite methods to hash keys

spec/github/paged_request_spec.rb

@@ -6,7 +6,7 @@
   let(:current_api) { Github::Client::Repos.new }
   let(:path) { "/repositories/"}
   let(:klass) {
-    klass = Class.new do
+    Class.new do
       include Github::PagedRequest
     end
   }

spec/github/validations/required_spec.rb

@@ -6,7 +6,7 @@
 describe Github::Validations::Required do
 
   let(:validator) {
-    klaz = Class.new.extend(described_class)
+    Class.new.extend(described_class)
   }
 
   context '#assert_required_keys' do