jekyll-minibundle 1.3.0 → 1.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7676c0eba42d8141ef77c3fe778c401945f6bb74
-  data.tar.gz: a22db95aa1900aa83c1ef4f1b785bf10efde371a
+  metadata.gz: e710ae4041aa12d52354bcfe4d26cede0a7f8c11
+  data.tar.gz: d50a377c564e3186c790203e8c90d71a02c66cf8
 SHA512:
-  metadata.gz: 596712cfd61a61dd4f0524c809bfb3cf967fb5cbe451f59746f76e44efd3af907b8770db6928dda57c6c0c6e086f5ab2d1c20c4acd51ab3a392faa81b363b327
-  data.tar.gz: 5deded83c36f011098f81208fc548fbc3e8616581a208a0df4fe31695a5ca76eb0944efed20ed348ab57daccb29827f82a635c5e702684ef5ae19aa1e6977a90
+  metadata.gz: dadbb64b20d2000b3f02285ed43b9d69ebcfd46201f62a034ec08cd7d24cf645cd2415265a05af1d7dc0d6c65438cf61ed1c657cd4abd02f53daff0a221d76c7
+  data.tar.gz: b214ee8464dc1f7ac95da50cbbf3635491fc9f990b167add78ef6f956e421770d9b8391d4b2e9572ce8f735c64eb9c796a4e08326f6b8c7d4d21c01deb851f78

data/CHANGELOG.md

@@ -1,8 +1,16 @@
+# 1.4.0 / 2013-12-27
+
+* Fix bug causing exception to be thrown when `ministamp` or
+  `minibundle` is called twice with same asset source argument. Allow
+  handling asset source files that are already static files in Jekyll
+  (remove the restriction introduced in 1.3.0). (@agrigg)
+
 # 1.3.0 / 2013-12-25
 
 * Disallow handling asset source files that are already static files
-  in Jekyll (would potentially lead to inconsistencies in watch mode,
-  see "Jekyll static file restriction" in README.md)
+  in Jekyll. Otherwise, we would potentially get to inconsistencies in
+  Jekyll's watch mode. See "Jekyll static file restriction" in
+  README.md. (@agrigg)
 * Upgrade development dependencies
 
 # 1.2.0 / 2013-09-29

data/README.md

@@ -1,12 +1,12 @@
 # Jekyll Minibundle plugin
 
-A straightforward asset bundling plugin for
-[Jekyll](http://jekyllrb.com/), utilizing external minification tool
-of your choice. Provides asset concatenation for bundling and asset
-fingerprinting with MD5 digest for cache busting.  No other runtime
-dependencies besides the minification tool (not even other gems).
+A straightforward asset bundling plugin for [Jekyll][Jekyll],
+utilizing external minification tool of your choice. It provides asset
+concatenation for bundling and asset fingerprinting with MD5 digest
+for cache busting. There are no other runtime dependencies besides the
+minification tool (not even other gems).
 
-Tested with Ruby MRI 1.9.3 and 2.0.0. Ruby 1.8 is *not* supported.
+Tested with Ruby MRI 1.9.3, 2.0, and 2.1. Ruby 1.8 is *not* supported.
 
 The plugin works with Jekyll's watch (auto-regeneration) mode.
 
@@ -53,6 +53,9 @@ this line:
 require 'jekyll/minibundle'
 ```
 
+You must allow Jekyll to use custom plugins. In
+[Jekyll's configuration][JekyllConf], do not enable `safe` setting.
+
 ## Asset fingerprinting
 
 Asset fingerprinting is intended to be used together with
@@ -135,27 +138,26 @@ attributes:
 And then specify the command for launching bundling in
 `$JEKYLL_MINIBUNDLE_CMD_CSS` environment variable.
 
-## Jekyll static file restriction
+## Recommended directory layout
 
-The plugin disallows using asset source files already known to Jekyll
-as static files. This is because the plugin operates by adding asset
-files as special static files to Jekyll, allowing the plugin work
-nicely with Jekyll's watch (auto-regeneration) mode.
+It's recommended that you exclude the files you use as asset sources
+from Jekyll itself. Otherwise, you end up with duplicate files in the
+output directory.
 
-For example, the following snippet causes the plugin to throw an
-exception if `assets/src.css` is not excluded from Jekyll:
+For example, in the following snippet we're using `assets/src.css` as
+asset source to `ministamp` tag:
 
 ``` html
 <link href="{% ministamp assets/src.css assets/dest.css %}" rel="stylesheet" media="screen, projection">
 ```
 
-If the restriction would not exist, there would be both `src.css` and
-`dest-<md5-hash>.css` files in `_site/assets/` output directory, which
-you probably wouldn't want to happen.
+By default, Jekyll includes this file to the output directory. As a
+result, there will be both `src.css` and `dest-<md5>.css` files in
+`_site/assets/` directory, which you probably do not want.
 
-By default, Jekyll excludes directories beginning with underscore
-character (`_`), so you might use the following directory layout for
-assets:
+In order to avoid this, exclude the asset source file from
+Jekyll. Because Jekyll excludes directories beginning with underscore
+character (`_`), consider the following directory layout:
 
 * `_assets/` for JS and CSS assets handled by the plugin that are in
   version control
@@ -163,8 +165,8 @@ assets:
   are not in version control (for example, Compass output files)
 * `assets/` for images and other assets handled by Jekyll directly
 
-See [Jekyll configuration](http://jekyllrb.com/docs/configuration/)
-for more about excluding files and directories.
+See [Jekyll configuration][JekyllConf] for more about excluding files
+and directories.
 
 ## Development mode
 
@@ -186,3 +188,6 @@ See the contents of `test/fixture/site` directory.
 # License
 
 MIT. See `LICENSE.txt`.
+
+[Jekyll]: http://jekyllrb.com/
+[JekyllConf]: http://jekyllrb.com/docs/configuration/

data/RELEASING.txt

@@ -3,7 +3,7 @@ In order to release, do
 $EDITOR lib/jekyll/minibundle/version.rb  # update version number
 $EDITOR CHANGELOG.md                      # describe a summary of changes since last release
 git diff                                  # review the changes
-rake gem:build                            # check that tests pass
+bundle exec rake gem:build                # check that tests pass
 git add -p
 git commit -m 'Release <version>'
 git tag <version>

data/Rakefile

@@ -1,3 +1,4 @@
+require 'fileutils'
 require 'rake/clean'
 require 'shellwords'
 
@@ -12,6 +13,12 @@ def get_minibundle_env(overrides = {})
   }.merge(overrides)
 end
 
+def run_jekyll_in_fixture_site(command)
+  Dir.chdir 'test/fixture/site'
+  FileUtils.rm_rf '_site'
+  sh get_minibundle_env, "jekyll #{command}"
+end
+
 namespace :gem do
   gem_name = 'jekyll-minibundle'
 
@@ -46,12 +53,18 @@ task :test do
   sh env, cmd
 end
 
-desc 'Generate fixture site for debugging'
-task :debug do
-  Dir.chdir 'test/fixture/site'
-  sh(get_minibundle_env, 'jekyll')
-end
+namespace :fixture do
+  CLEAN.include 'test/fixture/site/_site'
 
-CLEAN.include 'test/fixture/site/_site'
+  desc 'Generate fixture site'
+  task :build do
+    run_jekyll_in_fixture_site 'build'
+  end
+
+  desc 'Generate fixture site in watch mode'
+  task :watch do
+    run_jekyll_in_fixture_site 'build --watch'
+  end
+end
 
 task :default => :test

data/lib/jekyll/minibundle/asset_bundle.rb

@@ -30,7 +30,7 @@ module Jekyll::Minibundle
 
     if defined? ::Jekyll.logger  # introduced in Jekyll 1.0.0
       def log(msg)
-        ::Jekyll.logger.info 'Minibundle', msg
+        ::Jekyll.logger.info 'Minibundle:', msg
       end
     else
       def log(msg)

data/lib/jekyll/minibundle/asset_file_operations.rb

@@ -3,13 +3,9 @@ require 'fileutils'
 module Jekyll::Minibundle
   module AssetFileOperations
     def static_file!(site)
-      check_no_existing_static_file site.static_files
-      site.static_files << self
-    end
-
-    def check_no_existing_static_file(static_files)
-      found = static_files.find { |f| f.path == path }
-      raise "Minibundle cannot handle static file already handled by Jekyll: #{path}" if found
+      unless site.static_files.include? self
+        site.static_files << self
+      end
     end
 
     def write_destination(site_destination_dir)

data/lib/jekyll/minibundle/asset_file_paths.rb

@@ -21,7 +21,7 @@ module Jekyll::Minibundle
     end
 
     def modified?
-      last_mtime_of(path) != mtime
+      stamped_at != mtime
     end
   end
 end

data/lib/jekyll/minibundle/bundle_file.rb

@@ -7,14 +7,6 @@ module Jekyll::Minibundle
   class BundleFile
     include AssetFileOperations
 
-    def self.clear_cache
-      @@mtimes = {}
-      @@writes_after_rebundling = Hash.new false
-      @@asset_bundles = {}
-    end
-
-    clear_cache
-
     def initialize(config)
       @type = config['type']
       @site_source_dir = config['site_dir']
@@ -22,12 +14,22 @@ module Jekyll::Minibundle
       @assets = config['assets'].map { |asset_path| File.join asset_source_dir, "#{asset_path}.#{@type}" }
       @destination_path = config['destination_path']
       @attributes = config['attributes']
+      @stamped_at = nil
+      @is_modified = false
     end
 
     def markup
       # we must rebundle here, if at all, in order to make sure the
       # markup and generated file have the same fingerprint
-      rebundle_assets if modified?
+      if modified?
+        @stamped_at = mtime
+        @is_modified = true
+        @_asset_stamp = nil
+        asset_bundle.make_bundle
+      else
+        @is_modified = false
+      end
+
       AssetTagMarkup.make_markup @type, asset_destination_path, @attributes
     end
 
@@ -35,11 +37,6 @@ module Jekyll::Minibundle
       asset_bundle.path
     end
 
-    def check_no_existing_static_file(static_files)
-      existing = @assets & static_files.map(&:path)
-      raise "Minibundle cannot handle static file already handled by Jekyll: #{existing.first}" unless existing.empty?
-    end
-
     def asset_destination_path
       "#{@destination_path}-#{asset_stamp}.#{@type}"
     end
@@ -53,43 +50,26 @@ module Jekyll::Minibundle
     end
 
     def modified?
-      @@mtimes[asset_destination_canonical_path] != mtime
+      @stamped_at != mtime
     end
 
     def write(site_destination_dir)
-      if File.exists?(destination(site_destination_dir)) && destination_written_after_rebundling?
+      if File.exists?(destination(site_destination_dir)) && !@is_modified
         false
       else
         write_destination site_destination_dir
-        @@writes_after_rebundling[asset_destination_canonical_path] = true
         true
       end
     end
 
     private
 
-    def asset_destination_canonical_path
-      "#{@destination_path}.#{@type}"
-    end
-
     def asset_stamp
-      @asset_stamp ||= AssetStamp.from_file path
+      @_asset_stamp ||= AssetStamp.from_file path
     end
 
     def asset_bundle
-      @@asset_bundles[asset_destination_canonical_path] ||= AssetBundle.new(@type, @assets, @site_source_dir)
-    end
-
-    def rebundle_assets
-      p = asset_destination_canonical_path
-      @@writes_after_rebundling[p] = false
-      @@mtimes[p] = mtime
-      @asset_stamp = nil
-      asset_bundle.make_bundle
-    end
-
-    def destination_written_after_rebundling?
-      @@writes_after_rebundling[asset_destination_canonical_path]
+      @_asset_bundle ||= AssetBundle.new(@type, @assets, @site_source_dir)
     end
   end
 end

data/lib/jekyll/minibundle/development_file.rb

@@ -6,25 +6,20 @@ module Jekyll::Minibundle
     include AssetFileOperations
     include AssetFilePaths
 
-    @@mtimes = {}
-
-    attr_reader :asset_source_path, :asset_destination_dir, :asset_destination_basename
+    attr_reader :asset_source_path, :asset_destination_dir, :asset_destination_basename, :stamped_at
 
     def initialize(asset_source_path, asset_destination_path)
       @asset_source_path = asset_source_path
       @asset_destination_dir = File.dirname asset_destination_path
       @asset_destination_basename = File.basename asset_destination_path
-    end
-
-    def last_mtime_of(path)
-      @@mtimes[path]
+      @stamped_at = nil
     end
 
     def write(site_destination_dir)
       if destination_exists?(site_destination_dir) && !modified?
         false
       else
-        @@mtimes[path] = mtime
+        @stamped_at = mtime
         write_destination site_destination_dir
         true
       end

data/lib/jekyll/minibundle/mini_bundle_block.rb

@@ -1,7 +1,5 @@
 require 'yaml'
-require 'jekyll/minibundle/bundle_file'
-require 'jekyll/minibundle/development_file_collection'
-require 'jekyll/minibundle/environment'
+require 'jekyll/minibundle/asset_file_registry'
 
 module Jekyll::Minibundle
   class MiniBundleBlock < Liquid::Block
@@ -12,7 +10,8 @@ module Jekyll::Minibundle
 
     def render(context)
       site = context.registers[:site]
-      file = get_file get_current_config(YAML.load(super), site)
+      config = get_current_config YAML.load(super), site
+      file = AssetFileRegistry.bundle_file config
       file.static_file! site
       file.markup
     end
@@ -33,14 +32,6 @@ module Jekyll::Minibundle
         merge(user_config).
         merge({ 'type' => @type, 'site_dir' => site.source })
     end
-
-    def get_file(config)
-      if Environment.development?
-        DevelopmentFileCollection.new config
-      else
-        BundleFile.new config
-      end
-    end
   end
 end
 

data/lib/jekyll/minibundle/mini_stamp_tag.rb

@@ -1,4 +1,4 @@
-require 'jekyll/minibundle/stamp_file'
+require 'jekyll/minibundle/asset_file_registry'
 
 module Jekyll::Minibundle
   class MiniStampTag < Liquid::Tag
@@ -9,20 +9,10 @@ module Jekyll::Minibundle
 
     def render(context)
       site = context.registers[:site]
-      file = StampFile.new(File.join(site.source, @asset_source), @asset_destination, &get_basenamer)
+      file = AssetFileRegistry.stamp_file File.join(site.source, @asset_source), @asset_destination
       file.static_file! site
       file.markup
     end
-
-    private
-
-    def get_basenamer
-      if Environment.development?
-        ->(base, ext, _) { base + ext }
-      else
-        ->(base, ext, stamper) { "#{base}-#{stamper.call}#{ext}" }
-      end
-    end
   end
 end
 

data/lib/jekyll/minibundle/stamp_file.rb

@@ -7,9 +7,7 @@ module Jekyll::Minibundle
     include AssetFileOperations
     include AssetFilePaths
 
-    @@mtimes = {}
-
-    attr_reader :asset_source_path, :asset_destination_dir
+    attr_reader :asset_source_path, :asset_destination_dir, :stamped_at
 
     def initialize(asset_source_path, asset_destination_path, &basenamer)
       @basenamer = basenamer
@@ -17,7 +15,8 @@ module Jekyll::Minibundle
       @asset_destination_dir = File.dirname asset_destination_path
       @asset_destination_extension = File.extname asset_destination_path
       @asset_destination_base_prefix = File.basename(asset_destination_path)[0 .. -(@asset_destination_extension.size + 1)]
-      @was_modified = false
+      @stamped_at = nil
+      @is_modified = false
     end
 
     def markup
@@ -25,22 +24,18 @@ module Jekyll::Minibundle
       # to make sure the markup and generated file have the same
       # fingerprint
       if modified?
-        @asset_stamp = nil
-        @@mtimes[path] = mtime
-        @was_modified = true
+        @stamped_at = mtime
+        @is_modified = true
+        @_asset_stamp = nil
       else
-        @was_modified = false
+        @is_modified = false
       end
 
       asset_destination_path
     end
 
-    def last_mtime_of(path)
-      @@mtimes[path]
-    end
-
     def write(site_destination_dir)
-      if destination_exists?(site_destination_dir) && !@was_modified
+      if destination_exists?(site_destination_dir) && !@is_modified
         false
       else
         write_destination site_destination_dir
@@ -55,7 +50,7 @@ module Jekyll::Minibundle
     end
 
     def asset_stamp
-      @asset_stamp ||= AssetStamp.from_file(path)
+      @_asset_stamp ||= AssetStamp.from_file(path)
     end
   end
 end

data/lib/jekyll/minibundle/version.rb

@@ -1,5 +1,5 @@
 module Jekyll
   module Minibundle
-    VERSION = '1.3.0'
+    VERSION = '1.4.0'
   end
 end

data/test/fixture/site/_assets/scripts/app.js

@@ -1,4 +1,4 @@
-/* comment to be removed in bundling */
+/* asset file, handled by Minibundle */
 (function(root, dependency) {
   if (!dependency) throw new Error("missing dependency");
   root.app = {};

data/test/fixture/site/_assets/scripts/dependency.js

@@ -1,4 +1,4 @@
-/* comment to be removed in bundling */
+/* asset file, handled by Minibundle */
 (function(root) {
   root.dependency = {};
 })(window)

data/test/fixture/site/_assets/styles/common.css

@@ -1,2 +1,2 @@
-/* comment to be removed in bundling */
+/* asset file, handled by Minibundle */
 p { margin: 0; }

data/test/fixture/site/_assets/styles/reset.css

@@ -1,2 +1,2 @@
-/* comment to be removed in bundling */
+/* asset file, handled by Minibundle */
 html { margin: 0; }

data/test/fixture/site/_tmp/site.css

@@ -1,2 +1,3 @@
+/* asset file, handled by Minibundle */
 html { margin: 0; }
 p { margin: 2px; }

data/test/fixture/site/index.html

@@ -1,31 +1,4 @@
 ---
-layout: nil
+layout: default
+title: Index
 ---
-
-<!DOCTYPE html>
-<html>
-  <head>
-    <link rel="stylesheet" href="{% ministamp _tmp/site.css assets/screen.css %}" media="screen">
-    {% minibundle css %}
-    source_dir: _assets/styles
-    destination_path: assets/site
-    assets:
-      - reset
-      - common
-    attributes:
-      id: my-styles
-      media: projection
-    {% endminibundle %}
-  </head>
-  <body>
-    {% minibundle js %}
-    source_dir: _assets/scripts
-    destination_path: assets/site
-    assets:
-      - dependency
-      - app
-    attributes:
-      id: my-scripts
-    {% endminibundle %}
-  </body>
-</html>

data/test/integration/minibundle_development_mode_test.rb

@@ -45,7 +45,7 @@ module Jekyll::Minibundle::Test
 
         refute_equal File.read(destination), File.read(source)
 
-        generate_site :development
+        generate_site :development, clear_cache: false
 
         assert_equal File.read(destination), File.read(source)
       end
@@ -60,7 +60,7 @@ module Jekyll::Minibundle::Test
 
         refute_equal File.read(destination), File.read(source)
 
-        generate_site :development
+        generate_site :development, clear_cache: false
 
         assert_equal File.read(destination), File.read(source)
       end
@@ -77,8 +77,8 @@ module Jekyll::Minibundle::Test
         assert_equal 'assets/site/common.css', find_css_paths_from_index.last
         assert_equal 'assets/site/app.js', find_js_paths_from_index.last
 
-        find_and_gsub_in_file source_path('index.html'), 'destination_path: assets/site', 'destination_path: /assets/site'
-        generate_site :development
+        find_and_gsub_in_file source_path('_layouts/default.html'), 'destination_path: assets/site', 'destination_path: /assets/site'
+        generate_site :development, clear_cache: false
 
         assert File.exists?(expected_css_path)
         assert File.exists?(expected_js_path)
@@ -102,12 +102,12 @@ module Jekyll::Minibundle::Test
         expected_js_path = destination_path JS_BUNDLE_DESTINATION_PATH, 'app.js'
         org_mtime = mtime_of expected_js_path
         ensure_file_mtime_changes { File.write source_path(JS_BUNDLE_SOURCE_DIR, 'dependency.js'), '(function() {})()' }
-        generate_site :development
+        generate_site :development, clear_cache: false
 
         assert_equal org_mtime, mtime_of(expected_js_path)
 
         ensure_file_mtime_changes { FileUtils.touch 'index.html' }
-        generate_site :development
+        generate_site :development, clear_cache: false
 
         assert_equal org_mtime, mtime_of(expected_js_path)
       end

data/test/integration/minibundle_production_mode_test.rb

@@ -128,8 +128,8 @@ module Jekyll::Minibundle::Test
         assert_equal EXPECTED_CSS_BUNDLE_PATH, find_css_path_from_index
         assert_equal EXPECTED_JS_BUNDLE_PATH, find_js_path_from_index
 
-        find_and_gsub_in_file source_path('index.html'), 'destination_path: assets/site', 'destination_path: /assets/site'
-        generate_site :production
+        find_and_gsub_in_file source_path('_layouts/default.html'), 'destination_path: assets/site', 'destination_path: /assets/site'
+        generate_site :production, clear_cache: false
 
         assert File.exists?(expected_css_path)
         assert File.exists?(expected_js_path)

data/test/integration/ministamp_test.rb

@@ -34,11 +34,16 @@ module Jekyll::Minibundle::Test
     def test_change_asset_file
       with_site do
         generate_site :production
+
         assert File.exists?(destination_path(STAMP_DESTINATION_FINGERPRINT_PATH))
-        File.write source_path(STAMP_SOURCE_PATH), 'h1 {}'
-        generate_site :production
+
+        ensure_file_mtime_changes { File.write source_path(STAMP_SOURCE_PATH), 'h1 {}' }
+        generate_site :production, clear_cache: false
+
         refute File.exists?(destination_path(STAMP_DESTINATION_FINGERPRINT_PATH))
+
         expected_new_path = 'assets/screen-0f5dbd1e527a2bee267e85007b08d2a5.css'
+
         assert_equal expected_new_path, find_css_path_from_index
         assert File.exists?(destination_path(expected_new_path))
       end
@@ -52,9 +57,8 @@ module Jekyll::Minibundle::Test
         assert File.exists?(expected_path)
         assert_equal STAMP_DESTINATION_FINGERPRINT_PATH, find_css_path_from_index
 
-        find_and_gsub_in_file(source_path('index.html'), %r{assets/screen.css}, '/\0')
-
-        generate_site :production
+        find_and_gsub_in_file(source_path('_layouts/default.html'), %r{assets/screen.css}, '/\0')
+        generate_site :production, clear_cache: false
 
         assert File.exists?(expected_path)
         assert_equal "/#{STAMP_DESTINATION_FINGERPRINT_PATH}", find_css_path_from_index
@@ -67,12 +71,12 @@ module Jekyll::Minibundle::Test
         expected_path = destination_path STAMP_DESTINATION_FINGERPRINT_PATH
         org_mtime = mtime_of expected_path
         ensure_file_mtime_changes { File.write source_path(JS_BUNDLE_SOURCE_DIR, 'dependency.js'), '(function() {})()' }
-        generate_site :production
+        generate_site :production, clear_cache: false
 
         assert_equal org_mtime, mtime_of(expected_path)
 
         ensure_file_mtime_changes { FileUtils.touch 'index.html' }
-        generate_site :production
+        generate_site :production, clear_cache: false
 
         assert_equal org_mtime, mtime_of(expected_path)
       end

data/test/support/fixture_config.rb

@@ -2,7 +2,7 @@ module Jekyll::Minibundle::Test
   module FixtureConfig
     STAMP_SOURCE_PATH = '_tmp/site.css'
     STAMP_DESTINATION_PATH = 'assets/screen.css'
-    STAMP_DESTINATION_FINGERPRINT_PATH = 'assets/screen-390be921ee0eff063817bb5ef2954300.css'
+    STAMP_DESTINATION_FINGERPRINT_PATH = 'assets/screen-d57c1404fe726e66d57128a1bd190cbb.css'
 
     CSS_BUNDLE_DESTINATION_PATH = 'assets/site'
     CSS_BUNDLE_SOURCE_DIR = '_assets/styles'

data/test/support/test_case.rb

@@ -104,7 +104,7 @@ module Jekyll::Minibundle::Test
     end
 
     def _generate_site(test_options)
-      BundleFile.clear_cache if test_options[:clear_cache]
+      AssetFileRegistry.clear if test_options[:clear_cache]
 
       capture_io do
         Jekyll::Site.new(Jekyll.configuration(TestCase.site_generation_jekyll_options)).process

data/test/unit/bundle_file_test.rb

@@ -7,8 +7,6 @@ module Jekyll::Minibundle::Test
     include FixtureConfig
 
     def test_consistent_fingerprint_in_file_and_markup
-      BundleFile.clear_cache
-
       with_site do
         with_env 'JEKYLL_MINIBUNDLE_CMD_JS' => cmd_to_remove_comments_and_count do
           bundle_file = BundleFile.new({

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: jekyll-minibundle
 version: !ruby/object:Gem::Version
-  version: 1.3.0
+  version: 1.4.0
 platform: ruby
 authors:
 - Tuomas Kareinen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-12-25 00:00:00.000000000 Z
+date: 2013-12-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: jekyll