Merge pull request #1941 from ericproulx/frozen_string_literal

frozen_string_literal: true

Author: dblock

.rubocop.yml

@@ -1,5 +1,5 @@
 AllCops:
-  TargetRubyVersion: 2.1
+  TargetRubyVersion: 2.4
   Include:
     - Dangerfile
     - gemfiles/*.gemfile

.rubocop_todo.yml

@@ -1,6 +1,6 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2018-12-20 21:38:24 +0000 using RuboCop version 0.51.0.
+# on 2019-12-09 18:07:42 +0100 using RuboCop version 0.51.0.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
@@ -26,41 +26,47 @@ Lint/RescueWithoutErrorClass:
   Exclude:
     - 'lib/grape/validations/validators/coerce.rb'
 
-# Offense count: 47
+# Offense count: 1
+# Cop supports --auto-correct.
+Lint/UnneededRequireStatement:
+  Exclude:
+    - 'lib/grape.rb'
+
+# Offense count: 46
 Metrics/AbcSize:
   Max: 44
 
-# Offense count: 4
+# Offense count: 7
 # Configuration parameters: CountComments, ExcludedMethods.
 Metrics/BlockLength:
   Max: 182
 
-# Offense count: 9
+# Offense count: 10
 # Configuration parameters: CountComments.
 Metrics/ClassLength:
   Max: 302
 
-# Offense count: 31
+# Offense count: 30
 Metrics/CyclomaticComplexity:
   Max: 14
 
-# Offense count: 1234
+# Offense count: 1300
 # Configuration parameters: AllowHeredoc, AllowURI, URISchemes, IgnoreCopDirectives, IgnoredPatterns.
 # URISchemes: http, https
 Metrics/LineLength:
   Max: 215
 
-# Offense count: 57
+# Offense count: 60
 # Configuration parameters: CountComments.
 Metrics/MethodLength:
-  Max: 34
+  Max: 33
 
 # Offense count: 12
 # Configuration parameters: CountComments.
 Metrics/ModuleLength:
   Max: 220
 
-# Offense count: 21
+# Offense count: 22
 Metrics/PerceivedComplexity:
   Max: 14
 
@@ -74,8 +80,40 @@ Naming/FileName:
     - 'Guardfile'
     - 'Rakefile'
 
+# Offense count: 1
+# Cop supports --auto-correct.
+Performance/RegexpMatch:
+  Exclude:
+    - 'lib/grape/middleware/versioner/path.rb'
+
 # Offense count: 2
 # Configuration parameters: SupportedStyles.
 # SupportedStyles: annotated, template
 Style/FormatStringToken:
   EnforcedStyle: template
+
+# Offense count: 1
+# Cop supports --auto-correct.
+# Configuration parameters: AutoCorrect, EnforcedStyle, SupportedStyles.
+# SupportedStyles: predicate, comparison
+Style/NumericPredicate:
+  Exclude:
+    - 'spec/**/*'
+    - 'lib/grape/middleware/formatter.rb'
+
+# Offense count: 12
+# Cop supports --auto-correct.
+# Configuration parameters: ConvertCodeThatCanStartToReturnNil.
+Style/SafeNavigation:
+  Exclude:
+    - 'lib/grape/api/instance.rb'
+    - 'lib/grape/dsl/desc.rb'
+    - 'lib/grape/dsl/helpers.rb'
+    - 'lib/grape/dsl/inside_route.rb'
+    - 'lib/grape/dsl/routing.rb'
+    - 'lib/grape/endpoint.rb'
+    - 'lib/grape/middleware/error.rb'
+    - 'lib/grape/middleware/versioner/accept_version_header.rb'
+    - 'lib/grape/middleware/versioner/header.rb'
+    - 'lib/grape/middleware/versioner/param.rb'
+    - 'lib/grape/middleware/versioner/path.rb'

Appraisals

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 appraise 'rails-5' do
   gem 'rails', '5.2.1'
 end

CHANGELOG.md

@@ -1,2 +1,4 @@
+# frozen_string_literal: true
+
 danger.import_dangerfile(gem: 'ruby-grape-danger')
 toc.check

Dangerfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # when changing this file, run appraisal install ; rubocop -a gemfiles/*.gemfile
 
 source 'https://rubygems.org'

Gemfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 guard :rspec, all_on_start: true, cmd: 'bundle exec rspec' do
   watch(%r{^spec/.+_spec\.rb$})
   watch(%r{^lib/(.+)\.rb$})     { |m| "spec/#{m[1]}_spec.rb" }

Guardfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rubygems'
 require 'bundler'
 Bundler.setup :default, :test, :development

Rakefile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'grape'
 require 'benchmark/ips'
 

benchmark/nested_params.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 $LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
 require 'grape'
 require 'benchmark/ips'