Update rubocop requirement from ~> 0.50.0 to ~> 0.72.0 (#140)

Update rubocop requirement from ~> 0.50.0 to ~> 0.72.0

Co-authored-by: null <27856297+dependabot-preview[bot]@users.noreply.github.com>
Co-authored-by: Marc Riera <mrc2407@gmail.com>

.rubocop.yml

@@ -1,7 +1,10 @@
 inherit_from: .rubocop_todo.yml
 
 AllCops:
-  TargetRubyVersion: 2.2
+  TargetRubyVersion: 2.3
 
 Metrics/BlockLength:
   ExcludedMethods: [it, describe]
+
+Style/FrozenStringLiteralComment:
+  Enabled: false

.rubocop_todo.yml

@@ -52,6 +52,6 @@ Style/DoubleNegation:
     - 'lib/hyperclient/link.rb'
 
 # Offense count: 1
-Style/MethodMissing:
+Style/MethodMissingSuper:
   Exclude:
     - 'lib/hyperclient/collection.rb'

Gemfile

@@ -16,7 +16,7 @@ end
 
 group :development, :test do
   gem 'rake'
-  gem 'rubocop', '~> 0.50.0', require: false
+  gem 'rubocop', '~> 0.72.0', require: false
   gem 'simplecov', require: false
 end
 

Rakefile

@@ -1,4 +1,5 @@
 #!/usr/bin/env rake
+
 require 'rubygems'
 require 'bundler'
 Bundler.setup :default, :test, :development

hyperclient.gemspec

@@ -1,5 +1,4 @@
-
-require File.expand_path('../lib/hyperclient/version', __FILE__)
+require File.expand_path('lib/hyperclient/version', __dir__)
 
 Gem::Specification.new do |gem|
   gem.authors       = ['Oriol Gual']
@@ -14,10 +13,10 @@ Gem::Specification.new do |gem|
   gem.require_paths = ['lib']
   gem.version       = Hyperclient::VERSION
 
+  gem.add_dependency 'addressable'
   gem.add_dependency 'faraday', '>= 0.9.0'
-  gem.add_dependency 'faraday_middleware'
+  gem.add_dependency 'faraday-digestauth'
   gem.add_dependency 'faraday_hal_middleware'
-  gem.add_dependency 'addressable'
+  gem.add_dependency 'faraday_middleware'
   gem.add_dependency 'net-http-digest_auth'
-  gem.add_dependency 'faraday-digestauth'
 end

lib/hyperclient/curie.rb

@@ -41,7 +41,8 @@ def inspect
     # Returns a new expanded url.
     def expand(rel)
       return rel unless rel && templated?
-      href.gsub('{rel}', rel) if href
+
+      href&.gsub('{rel}', rel)
     end
   end
 end

lib/hyperclient/entry_point.rb

@@ -57,6 +57,7 @@ def connection(options = {}, &block)
       @faraday_options ||= options.dup
       if block_given?
         raise ConnectionAlreadyInitializedError if @connection
+
         @faraday_block = if @faraday_options.delete(:default) == false
                            block
                          else
@@ -75,6 +76,7 @@ def connection(options = {}, &block)
     # Returns a Hash.
     def headers
       return @connection.headers if @connection
+
       @headers ||= default_headers
     end
 
@@ -83,6 +85,7 @@ def headers
     # value    - A Hash containing headers to include with every API request.
     def headers=(value)
       raise ConnectionAlreadyInitializedError if @connection
+
       @headers = value
     end
 
@@ -98,6 +101,7 @@ def faraday_options
     # value    - A Hash containing options to pass to Faraday
     def faraday_options=(value)
       raise ConnectionAlreadyInitializedError if @connection
+
       @faraday_options = value
     end
 
@@ -113,6 +117,7 @@ def faraday_block
     # value    - A Proc accepting a Faraday::Connection.
     def faraday_block=(value)
       raise ConnectionAlreadyInitializedError if @connection
+
       @faraday_block = value
     end
 

lib/hyperclient/link.rb

@@ -39,6 +39,7 @@ def _expand(uri_variables = {})
     # Public: Returns the url of the Link.
     def _url
       return @link['href'] unless _templated?
+
       @url ||= _uri_template.expand(@uri_variables || {}).to_s
     end
 
@@ -137,8 +138,10 @@ def method_missing(method, *args, &block)
     # This allows `api.posts` instead of `api._links.posts.embedded.posts`
     def delegate_method(method, *args, &block)
       return unless @key && _resource.respond_to?(@key)
+
       @delegate ||= _resource.send(@key)
-      return unless @delegate && @delegate.respond_to?(method.to_s)
+      return unless @delegate&.respond_to?(method.to_s)
+
       @delegate.send(method, *args, &block)
     end
 

lib/hyperclient/resource.rb

@@ -55,11 +55,11 @@ def inspect
     end
 
     def _success?
-      _response && _response.success?
+      _response&.success?
     end
 
     def _status
-      _response && _response.status
+      _response&.status
     end
 
     def [](name)