Adds frozen_string_literal: true

controlshift · Feb 10, 2016 · 1b0f1de · 1b0f1de
1 parent 81f5cc1
commit 1b0f1de
Show file tree

Hide file tree

Showing 32 changed files with 34 additions and 18 deletions.
diff --git a/.rubocop.yml b/.rubocop.yml
@@ -18,6 +18,9 @@ Style/EmptyLineBetweenDefs:
 Style/Encoding:
   EnforcedStyle: when_needed
 
+Style/FrozenStringLiteralComment:
+  EnforcedStyle: always
+
 Style/HashSyntax:
   EnforcedStyle: hash_rockets
 
@@ -34,8 +37,7 @@ Style/MultilineOperationIndentation:
 # IMHO `%r{foo/bar}` looks way more cleaner than `/foo\/bar/`.
 # Enabling this cop also makes Guardfile (which is full of pathname regexps)
 # look absolutley (style) inconsistent and terrible. Thus it should be on
-# developer's choice whenever to user `%r` or not. Just like we don't enforce
-# to use `["foo"]` over `%w(foo)` and so on.
+# developer's choice whenever to user `%r` or not.
 Style/RegexpLiteral:
   Enabled: false
 

diff --git a/Gemfile b/Gemfile
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 source "https://rubygems.org"
 
 group :test do

diff --git a/Rakefile b/Rakefile
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require "bundler/gem_tasks"
 
 require "rspec/core/rake_task"

diff --git a/bin/console b/bin/console
diff --git a/bin/setup b/bin/setup
@@ -1,7 +1,6 @@
 #!/bin/bash
+
 set -euo pipefail
 IFS=$'\n\t'
 
 bundle install
-
-# Do any other automated setup that you need to do here
diff --git a/lib/sidekiq/throttled.rb b/lib/sidekiq/throttled.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # 3rd party
 require "sidekiq"
 

diff --git a/lib/sidekiq/throttled/basic_fetch.rb b/lib/sidekiq/throttled/basic_fetch.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # stdlib
 require "thread"
 

diff --git a/lib/sidekiq/throttled/errors.rb b/lib/sidekiq/throttled/errors.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Sidekiq
   module Throttled
     # Generic class for Sidekiq::Throttled errors

diff --git a/lib/sidekiq/throttled/middleware.rb b/lib/sidekiq/throttled/middleware.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # internal
 require "sidekiq/throttled/registry"
 

diff --git a/lib/sidekiq/throttled/registry.rb b/lib/sidekiq/throttled/registry.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # internal
 require "sidekiq/throttled/strategy"
 

diff --git a/lib/sidekiq/throttled/strategy.rb b/lib/sidekiq/throttled/strategy.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # internal
 require "sidekiq/throttled/errors"
 require "sidekiq/throttled/strategy/concurrency"

diff --git a/lib/sidekiq/throttled/strategy/concurrency.rb b/lib/sidekiq/throttled/strategy/concurrency.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # internal
 require "sidekiq/throttled/strategy/script"
 

diff --git a/lib/sidekiq/throttled/strategy/script.rb b/lib/sidekiq/throttled/strategy/script.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Sidekiq
   module Throttled
     class Strategy

diff --git a/lib/sidekiq/throttled/strategy/threshold.rb b/lib/sidekiq/throttled/strategy/threshold.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # internal
 require "sidekiq/throttled/strategy/script"
 

diff --git a/lib/sidekiq/throttled/testing.rb b/lib/sidekiq/throttled/testing.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require "sidekiq/throttled/registry"
 
 RSpec.configure do |config|

diff --git a/lib/sidekiq/throttled/version.rb b/lib/sidekiq/throttled/version.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Sidekiq
   module Throttled
     # Gem version

diff --git a/lib/sidekiq/throttled/web.rb b/lib/sidekiq/throttled/web.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # stdlib
 require "pathname"
 

diff --git a/lib/sidekiq/throttled/web/stats.rb b/lib/sidekiq/throttled/web/stats.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Sidekiq
   module Throttled
     module Web

diff --git a/lib/sidekiq/throttled/worker.rb b/lib/sidekiq/throttled/worker.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # internal
 require "sidekiq/throttled/registry"
 

diff --git a/sidekiq-throttled.gemspec b/sidekiq-throttled.gemspec
@@ -1,4 +1,5 @@
 # coding: utf-8
+# frozen_string_literal: true
 
 lib = File.expand_path("../lib", __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)

diff --git a/spec/sidekiq/throttled/basic_fetch_spec.rb b/spec/sidekiq/throttled/basic_fetch_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require "sidekiq/throttled/basic_fetch"
 
 RSpec.describe Sidekiq::Throttled::BasicFetch, :sidekiq => :disabled do

diff --git a/spec/sidekiq/throttled/middleware_spec.rb b/spec/sidekiq/throttled/middleware_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require "sidekiq/throttled/middleware"
 
 RSpec.describe Sidekiq::Throttled::Middleware do

diff --git a/spec/sidekiq/throttled/registry_spec.rb b/spec/sidekiq/throttled/registry_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 RSpec.describe Sidekiq::Throttled::Registry do
   let(:threshold)   { { :threshold => { :limit => 1, :period => 1 } } }
   let(:concurrency) { { :concurrency => { :limit => 1 } } }

diff --git a/spec/sidekiq/throttled/strategy/concurrency_spec.rb b/spec/sidekiq/throttled/strategy/concurrency_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 RSpec.describe Sidekiq::Throttled::Strategy::Concurrency do
   subject(:strategy) { described_class.new :test, :limit => 5 }
 

diff --git a/spec/sidekiq/throttled/strategy/threshold_spec.rb b/spec/sidekiq/throttled/strategy/threshold_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 RSpec.describe Sidekiq::Throttled::Strategy::Threshold do
   subject(:strategy) { described_class.new :test, :limit => 5, :period => 10 }
 

diff --git a/spec/sidekiq/throttled/strategy_spec.rb b/spec/sidekiq/throttled/strategy_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 RSpec.describe Sidekiq::Throttled::Strategy do
   subject(:strategy) { described_class.new(:foo, **options) }
 

diff --git a/spec/sidekiq/throttled/web/stats_spec.rb b/spec/sidekiq/throttled/web/stats_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require "sidekiq/throttled/web/stats"
 
 RSpec.describe Sidekiq::Throttled::Web::Stats do

diff --git a/spec/sidekiq/throttled/web_spec.rb b/spec/sidekiq/throttled/web_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require "rack/test"
 
 require "sidekiq/throttled/web"

diff --git a/spec/sidekiq/throttled/worker_spec.rb b/spec/sidekiq/throttled/worker_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 RSpec.describe Sidekiq::Throttled::Worker do
   let(:working_class) { Class.new { include Sidekiq::Throttled::Worker } }
 

diff --git a/spec/sidekiq/throttled_spec.rb b/spec/sidekiq/throttled_spec.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 RSpec.describe Sidekiq::Throttled, :sidekiq => :disabled do
   describe ".setup!" do
     before do

diff --git a/spec/sidekiq_helper.rb b/spec/sidekiq_helper.rb
@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require "securerandom"
 
 require "sidekiq/testing"

diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb
@@ -1,4 +1,5 @@
 # coding: utf-8
+# frozen_string_literal: true
 
 require "simplecov"
 require "coveralls"