Merge pull request #52 from matthodan/fix-rubocop-offenses-2

Fix rubocop offenses (again)

.rubocop.yml

@@ -1,7 +1,9 @@
+AllCops:
+  TargetRubyVersion: 2.3
+
 # Do not count block length in spec files
 Metrics/BlockLength:
   ExcludedMethods: ['describe', 'it', 'context']
 
 MethodLength:
   Max: 15
-

.travis.yml

@@ -1,7 +1,6 @@
 language: ruby
 
 rvm:
-  - 2.2
   - 2.3
   - 2.4
   - 2.5

Gemfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source 'http://rubygems.org'
 
 group :test do

Rakefile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rake'
 require 'rake/testtask'
 

jekyll_asset_pipeline.gemspec

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require File.expand_path('lib/jekyll_asset_pipeline/version', __dir__)
 
 Gem::Specification.new do |s|
@@ -22,7 +24,7 @@ Gem::Specification.new do |s|
   s.homepage = 'https://github.com/matthodan/jekyll-asset-pipeline'
   s.license = 'MIT'
 
-  s.required_ruby_version = '>= 2.2.0'
+  s.required_ruby_version = '>= 2.3.0'
   s.rubygems_version = '2.2.2'
 
   # Runtime dependencies

lib/jekyll_asset_pipeline.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Stdlib dependencies
 require 'digest/md5'
 require 'fileutils'

lib/jekyll_asset_pipeline/asset.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module JekyllAssetPipeline
   # Holds an asset (file)
   class Asset

lib/jekyll_asset_pipeline/compressor.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module JekyllAssetPipeline
   # Base class for asset compressors
   # See https://github.com/matthodan/jekyll-asset-pipeline#asset-compression

lib/jekyll_asset_pipeline/converter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module JekyllAssetPipeline
   # Base class for asset converters
   # See https://github.com/matthodan/jekyll-asset-pipeline#asset-preprocessing

lib/jekyll_asset_pipeline/extensions/jekyll/site.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Jekyll
   # Contains overrides for the needed Jekyll:Site methods
   # The actual code is in JekyllAssetPipeline::JekyllSiteExtensions

lib/jekyll_asset_pipeline/extensions/jekyll/site_extensions.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module JekyllAssetPipeline
   # Contains overrides for the needed Jekyll:Site methods
   # Included in Jekyll::Site

lib/jekyll_asset_pipeline/extensions/liquid/asset_tag.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module JekyllAssetPipeline
   # This is a Liquid tag block extension
   # See documentation here:

lib/jekyll_asset_pipeline/extensions/liquid/asset_tags/css_asset_tag.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # This comment is needed, otherwise Rubocop complains because of the
 # register_tag below and a verbose comment is better than a :nodoc: :)
 module JekyllAssetPipeline

lib/jekyll_asset_pipeline/extensions/liquid/asset_tags/javascript_asset_tag.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # This comment is needed, otherwise Rubocop complains because of the
 # register_tag below and a verbose comment is better than a :nodoc: :)
 module JekyllAssetPipeline

lib/jekyll_asset_pipeline/extensions/liquid/liquid_block_extensions.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module JekyllAssetPipeline
   # Helper module used by JekyllAssetPipeline::AssetTag as well as
   # classed derived from it (Liquid tag block extensions)

lib/jekyll_asset_pipeline/extensions/ruby/subclass_tracking.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module JekyllAssetPipeline
   # Allows classes that extend this to return an array of their subclasses
   module SubclassTracking

lib/jekyll_asset_pipeline/pipeline.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module JekyllAssetPipeline
   # The pipeline itself, the run method is where it all happens
   # rubocop:disable ClassLength

lib/jekyll_asset_pipeline/template.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module JekyllAssetPipeline
   # Base class for the tag templates
   # See https://github.com/matthodan/jekyll-asset-pipeline#templates

lib/jekyll_asset_pipeline/templates/css_tag_template.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module JekyllAssetPipeline
   # Default output for CSS assets
   class CssTagTemplate < JekyllAssetPipeline::Template

lib/jekyll_asset_pipeline/templates/javascript_tag_template.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module JekyllAssetPipeline
   # Default output for JavaScript assets
   class JavaScriptTagTemplate < JekyllAssetPipeline::Template

lib/jekyll_asset_pipeline/templates/template_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module JekyllAssetPipeline
   # Contains helper methods used by the tag template classes
   module TemplateHelper

lib/jekyll_asset_pipeline/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module JekyllAssetPipeline
-  VERSION = '0.6.0'.freeze
+  VERSION = '0.6.0'
 end

spec/helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'coveralls'
 Coveralls.wear!
 

spec/helpers/extensions/ruby/module.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Allow us to test if a module extends another module/class
 #
 # For example:

spec/jekyll_asset_pipeline/asset_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require './spec/helper'
 
 module JekyllAssetPipeline

spec/jekyll_asset_pipeline/compressor_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require './spec/helper'
 
 module JekyllAssetPipeline

spec/jekyll_asset_pipeline/converter_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require './spec/helper'
 
 module JekyllAssetPipeline

spec/jekyll_asset_pipeline/extensions/jekyll/site_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require './spec/helper'
 
 module JekyllAssetPipeline

spec/jekyll_asset_pipeline/extensions/jekyll_site_extensions_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require './spec/helper'
 
 module JekyllAssetPipeline

spec/jekyll_asset_pipeline/extensions/liquid/asset_tag_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require './spec/helper'
 
 module JekyllAssetPipeline

spec/jekyll_asset_pipeline/extensions/liquid/asset_tags/css_asset_tag_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require './spec/helper'
 
 module JekyllAssetPipeline

spec/jekyll_asset_pipeline/extensions/liquid/asset_tags/java_script_asset_tag_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require './spec/helper'
 
 module JekyllAssetPipeline

spec/jekyll_asset_pipeline/extensions/liquid/liquid_block_extensions_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require './spec/helper'
 
 module JekyllAssetPipeline

spec/jekyll_asset_pipeline/extensions/ruby/subclass_tracking_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require './spec/helper'
 
 module JekyllAssetPipeline

spec/jekyll_asset_pipeline/integration_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require './spec/helper'
 
 describe 'Integration' do

spec/jekyll_asset_pipeline/pipeline_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require './spec/helper'
 
 # rubocop:disable ModuleLength

spec/jekyll_asset_pipeline/template_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require './spec/helper'
 
 # rubocop:disable ModuleLength

spec/jekyll_asset_pipeline/version_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require './spec/helper'
 
 module JekyllAssetPipeline

spec/resources/source/_plugins/jekyll_asset_pipeline.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module JekyllAssetPipeline
   class TestConverter < JekyllAssetPipeline::Converter
     def self.filetype