jekyll-assets 0.7.6 → 0.7.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 9f2003fb2e8cde02889613bc3abacfa0e0d80947
-  data.tar.gz: 96eb2b56cf5aa69236d5e09c4cb6df1f52355108
+  metadata.gz: 8c10ed0e80b3a4beea2ca55d84e3fb90d0308a70
+  data.tar.gz: 3a4da62d35bac89038896162f6d78d367fd1c869
 SHA512:
-  metadata.gz: 2eab6482a241e44ebb84d4791e00e3bdbf8f1449147acbc7f426cb48f3cedba75f94a0d3dd54b61e0cbfbfc6ff1a43b9601a8339f5e5b906ef061a9f99656265
-  data.tar.gz: 054766f50e413169041b8dcda51f8e58f83c15c60ef213505225048d17c4d99859883e784672ad3fb82f28de520d16493a6fc66cb28cba14695a86e5fdc9334c
+  metadata.gz: f70e33d032f3aca16cb6d64ade3de4fb52138d46f0592be6e6f389bcc81085deb704dff05a9374ec570a78ea62c9835dc52b0f3e7d51ea712053fc00e2c8079e
+  data.tar.gz: 2bb58025391bef810dec0a607ef783c7899b332bb50b37773f73fdc88864bfee3d10781893a0e65f2cb74f962b62ffed6e338e211da9520f5249c906c485b3a3

data/.rubocop.yml

@@ -0,0 +1,48 @@
+# Avoid long parameter lists
+ParameterLists:
+  Max: 3
+  CountKeywordArgs: true
+
+MethodLength:
+  CountComments: false
+  Max: 15
+
+ClassLength:
+  CountComments: false
+  Max: 100
+
+CyclomaticComplexity:
+  Max: 6
+
+EmptyLineBetweenDefs:
+  AllowAdjacentOneLineDefs: true
+
+BlockNesting:
+  Max: 3
+
+HashSyntax:
+  EnforcedStyle: hash_rockets
+
+Encoding:
+  Enabled: false
+
+StringLiterals:
+  EnforcedStyle: double_quotes
+
+BracesAroundHashParameters:
+  Enabled: false
+
+Documentation:
+  Enabled: false
+
+# Not all trivial readers/writers can be defined with attr_* methods
+TrivialAccessors:
+  Enabled: false
+
+# New lambda syntax is UGLY, don't enforce it
+Lambda:
+  Enabled: false
+
+AllCops:
+  Includes:
+    - Gemfile

data/Gemfile

@@ -1,11 +1,11 @@
-source 'https://rubygems.org'
+source "https://rubygems.org"
 
+gem "rubocop"
 
 group :test do
-  gem 'coveralls', :require => false
-  gem 'simplecov', :require => false
+  gem "coveralls", :require => false
+  gem "simplecov", :require => false
 end
 
-
 # Specify your gem's dependencies in jekyll-assets.gemspec
 gemspec

data/Rakefile

@@ -1,7 +1,9 @@
 require "bundler/gem_tasks"
-require "rspec/core/rake_task"
 
+require "rspec/core/rake_task"
 RSpec::Core::RakeTask.new
 
-task :default => :spec
-task :test => :spec
+require "rubocop/rake_task"
+Rubocop::RakeTask.new
+
+task :default => [:spec, :rubocop]

data/lib/jekyll-assets/bootstrap.rb

@@ -1,7 +1,6 @@
-require "sprockets"
-
-
-bootstrap = Gem::Specification.find_by_name("bootstrap-sass").gem_dir
-%w(fonts javascripts stylesheets).each do |asset|
-  Sprockets.append_path File.join(bootstrap, 'vendor', 'assets', asset)
-end
+require "sprockets"
+
+bootstrap = Gem::Specification.find_by_name("bootstrap-sass").gem_dir
+%w(fonts javascripts stylesheets).each do |asset|
+  Sprockets.append_path File.join(bootstrap, "vendor", "assets", asset)
+end

data/lib/jekyll-assets/bourbon.rb

@@ -1,5 +1,4 @@
 require "sprockets"
 
-
 bourbon_root = Gem::Specification.find_by_name("bourbon").gem_dir
 Sprockets.append_path File.join(bourbon_root, "app", "assets", "stylesheets")

data/lib/jekyll-assets/compass.rb

@@ -1,9 +1,7 @@
 require "compass"
 require "sprockets"
 
-
 Compass::Frameworks::ALL.each do |fw|
-  if (path = fw.stylesheets_directory)
-    Sprockets.append_path path
-  end
+  path = fw.stylesheets_directory
+  Sprockets.append_path path if path
 end

data/lib/jekyll-assets/neat.rb

@@ -1,5 +1,4 @@
 require "sprockets"
 
-
 neat_root = Gem::Specification.find_by_name("neat").gem_dir
 Sprockets.append_path File.join(neat_root, "app", "assets", "stylesheets")

data/lib/jekyll/assets_plugin/asset_path.rb

@@ -1,26 +1,21 @@
 module Jekyll
   module AssetsPlugin
     class AssetPath
-
       attr_writer :anchor, :query
 
-
-      def initialize asset
+      def initialize(asset)
         asset.bundle!
         @asset = asset
       end
 
-
       def cachebust
         @cachebust ||= @asset.site.assets_config.cachebust
       end
 
-
       def path
         :hard == cachebust && @asset.digest_path || @asset.logical_path
       end
 
-
       def query
         query = []
 
@@ -30,16 +25,13 @@ module Jekyll
         "?#{query.join '&'}" unless query.empty?
       end
 
-
       def anchor
         "##{@anchor}" if @anchor
       end
 
-
       def to_s
         "#{@asset.site.assets_config.baseurl}/#{path}#{query}#{anchor}"
       end
-
     end
   end
 end

data/lib/jekyll/assets_plugin/configuration.rb

@@ -1,27 +1,28 @@
 # stdlib
 require "ostruct"
 
-
 module Jekyll
   module AssetsPlugin
     class Configuration
       DEFAULTS = {
         :dirname        => "assets",
-        :sources        => %w{_assets/javascripts _assets/stylesheets _assets/images},
+        :sources        => %w{ _assets/javascripts
+                               _assets/stylesheets
+                               _assets/images },
         :js_compressor  => nil,
         :css_compressor => nil,
         :cachebust      => :hard,
         :cache          => false,
         :gzip           => %w{ text/css application/javascript },
-        :debug          => false
+        :debug          => false,
+        :version        => 1
       }.freeze
 
-
-      def initialize config = {}
+      def initialize(config = {})
         @data = OpenStruct.new DEFAULTS.merge(config)
 
-        @data.sources  = [ @data.sources ] if @data.sources.is_a? String
-        @data.dirname  = @data.dirname.gsub(/^\/+|\/+$/, "")
+        @data.sources  = [@data.sources] if @data.sources.is_a? String
+        @data.dirname  = @data.dirname.gsub(%r{^/+|/+$}, "")
 
         compress = OpenStruct.new @data.compress
 
@@ -30,62 +31,55 @@ module Jekyll
         @data.cache           ||= @data.cache_assets
 
         # if baseurl not given - autoguess base on dirname
-        @data.baseurl ||= "/#{@data.dirname}/".squeeze '/'
+        @data.baseurl ||= "/#{@data.dirname}/".squeeze "/"
       end
 
-
       def baseurl
         @data.baseurl.chomp "/"
       end
 
-
       def js_compressor
         compressor @data.js_compressor
       end
 
-
       def css_compressor
         compressor @data.css_compressor
       end
 
-
       def cachebust
         return :none if none?(@data.cachebust)
-        return @data.cachebust.to_sym if @data.cachebust.to_s =~ /^(soft|hard)$/
-        raise "Unknown cachebust strategy: #{@data.cachebust}"
+        return @data.cachebust.to_sym if @data.cachebust.to_s[/^(soft|hard)$/]
+        fail "Unknown cachebust strategy: #{@data.cachebust}"
       end
 
-
       def cache_assets?
         !!@data.cache
       end
 
-
       def cache_path
         @data.cache.is_a?(String) ? @data.cache : ".jekyll-assets-cache"
       end
 
-
       def gzip
         return @data.gzip if @data.gzip.is_a? Array
         @data.gzip ? DEFAULTS[:gzip] : []
       end
 
+      def version
+        @data.version
+      end
 
-      def method_missing name, *args, &block
+      def method_missing(name, *args, &block)
         @data.send name, *args, &block
       end
 
+      private
 
-      protected
-
-
-      def none? val
+      def none?(val)
         val.to_s.empty? || "none" == val.to_s.downcase
       end
 
-
-      def compressor val
+      def compressor(val)
         none?(val) ? nil : val.to_sym
       end
     end

data/lib/jekyll/assets_plugin/environment.rb

@@ -1,26 +1,21 @@
 # stdlib
 require "pathname"
 
-
 # 3rd-party
 require "sprockets"
 
-
 module Jekyll
   module AssetsPlugin
     class Environment < Sprockets::Environment
-
       class AssetNotFound < StandardError
-        def initialize path
+        def initialize(path)
           super "Couldn't find file '#{path}'"
         end
       end
 
-
       attr_reader :site
 
-
-      def initialize site
+      def initialize(site)
         super site.source
 
         @site = site
@@ -43,16 +38,13 @@ module Jekyll
         context_class.send :include, Patches::ContextPatch
       end
 
-
       def cache_path
         Pathname.new(@site.source).join @site.assets_config.cache_path
       end
 
-
-      def find_asset path, *args
-        super or raise AssetNotFound, path
+      def find_asset(path, *args)
+        super || fail(AssetNotFound, path)
       end
-
     end
   end
 end

data/lib/jekyll/assets_plugin/filters.rb

@@ -1,11 +1,9 @@
 # internal
-require 'jekyll/assets_plugin/renderer'
-
+require "jekyll/assets_plugin/renderer"
 
 module Jekyll
   module AssetsPlugin
     module Filters
-
       %w{ asset asset_path image javascript stylesheet }.each do |name|
         module_eval <<-RUBY, __FILE__, __LINE__
         def #{name} path                    # def stylesheet logical_path
@@ -14,10 +12,8 @@ module Jekyll
         end                                 # end
         RUBY
       end
-
     end
   end
 end
 
-
 Liquid::Template.register_filter Jekyll::AssetsPlugin::Filters

data/lib/jekyll/assets_plugin/patches/asset_patch.rb

@@ -1,66 +1,53 @@
 # 3rd-party
 require "sprockets"
 
-
 module Jekyll
   module AssetsPlugin
     module Patches
       module AssetPatch
-
-        def self.included base
+        def self.included(base)
           base.send :extend, ClassMethods
           base.send :include, InstanceMethods
         end
 
-
         module ClassMethods
-
           def mtimes
-            @mtimes ||= Hash.new
+            @mtimes ||= {}
           end
-
         end
 
-
         module InstanceMethods
-
           attr_reader :site
 
-
           def jekyll_assets
             []
           end
 
-
           def bundle!
             site.bundle_asset! self if site
             self
           end
 
-
-          def destination dest
+          def destination(dest)
             File.join dest, site.assets_config.dirname, filename
           end
 
-
           def filename
             case cachebust = site.assets_config.cachebust
             when :none, :soft then logical_path
             when :hard        then digest_path
-            else raise "Unknown cachebust strategy: #{cachebust.inspect}"
+            else fail "Unknown cachebust strategy: #{cachebust.inspect}"
             end
           end
 
-
           def modified?
             self.class.mtimes[pathname.to_s] != mtime.to_i
           end
 
-
-          def write dest
+          def write(dest)
             dest_path = destination dest
 
-            return false if File.exist?(dest_path) and !modified?
+            return false if File.exist?(dest_path) && !modified?
             self.class.mtimes[pathname.to_s] = mtime.to_i
 
             write_to dest_path
@@ -69,17 +56,13 @@ module Jekyll
             true
           end
 
-
           def gzip?
             site.assets_config.gzip.include? content_type
           end
-
         end
-
       end
     end
   end
 end
 
-
 Sprockets::Asset.send :include, Jekyll::AssetsPlugin::Patches::AssetPatch