OAuth2 - PKCE | CI

__review__ * Remove Ruby 2.3 frozen string magic comments as requested because it is out of scope for this feature. * Add an exception to `rubocop.yml` so that the missing magic comment is not a CI failure.
omniauth · Aug 10, 2020 · 9ef9e5b · 9ef9e5b
1 parent a7b3b73
commit 9ef9e5b
Show file tree

Hide file tree

Showing 9 changed files with 4 additions and 16 deletions.
diff --git a/.rubocop.yml b/.rubocop.yml
@@ -52,6 +52,9 @@ Style/DoubleNegation:
 Style/ExpandPathArguments:
   Enabled: false
 
+Style/FrozenStringLiteralComment:
+  Enabled: false
+
 Style/HashSyntax:
   EnforcedStyle: hash_rockets
 

diff --git a/Gemfile b/Gemfile
@@ -1,5 +1,3 @@
-# frozen_string_literal: true
-
 source "https://rubygems.org"
 
 gem "rake", "~> 12.0"

diff --git a/Rakefile b/Rakefile
@@ -1,5 +1,4 @@
 #!/usr/bin/env rake
-# frozen_string_literal: true
 
 require "bundler/gem_tasks"
 require "rspec/core/rake_task"

diff --git a/lib/omniauth-oauth2.rb b/lib/omniauth-oauth2.rb
@@ -1,4 +1,2 @@
-# frozen_string_literal: true
-
 require "omniauth-oauth2/version"
 require "omniauth/strategies/oauth2"
diff --git a/lib/omniauth-oauth2/version.rb b/lib/omniauth-oauth2/version.rb
@@ -1,7 +1,5 @@
-# frozen_string_literal: true
-
 module OmniAuth
   module OAuth2
-    VERSION = "1.6.0"
+    VERSION = "1.6.0".freeze
   end
 end
diff --git a/lib/omniauth/strategies/oauth2.rb b/lib/omniauth/strategies/oauth2.rb
@@ -1,5 +1,3 @@
-# frozen_string_literal: true
-
 require "oauth2"
 require "omniauth"
 require "securerandom"

diff --git a/omniauth-oauth2.gemspec b/omniauth-oauth2.gemspec
@@ -1,5 +1,3 @@
-# frozen_string_literal: true
-
 lib = File.expand_path("../lib", __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require "omniauth-oauth2/version"

diff --git a/spec/helper.rb b/spec/helper.rb
@@ -1,5 +1,3 @@
-# frozen_string_literal: true
-
 $LOAD_PATH.unshift File.expand_path("..", __FILE__)
 $LOAD_PATH.unshift File.expand_path("../../lib", __FILE__)
 

diff --git a/spec/omniauth/strategies/oauth2_spec.rb b/spec/omniauth/strategies/oauth2_spec.rb
@@ -1,5 +1,3 @@
-# frozen_string_literal: true
-
 require "helper"
 
 describe OmniAuth::Strategies::OAuth2 do