appease Rubocop

Author: DirtyF

.rubocop.yml

@@ -1,2 +1,10 @@
 inherit_gem:
   jekyll: .rubocop.yml
+
+
+Metrics/BlockLength:
+  Exclude:
+    - test/**/*.rb
+Metrics/LineLength:
+  Exclude:
+    - test/**/*.rb

Gemfile

@@ -1,4 +1,4 @@
-source 'https://rubygems.org'
+source "https://rubygems.org"
 gemspec
 
 if ENV["GH_PAGES"]
@@ -8,4 +8,4 @@ elsif ENV["JEKYLL_VERSION"]
 end
 
 # Support for Ruby < 2.2.2 & activesupport
-gem "activesupport", "~> 4.2" if RUBY_VERSION < '2.2.2'
+gem "activesupport", "~> 4.2" if RUBY_VERSION < "2.2.2"

Rakefile

@@ -1,7 +1,7 @@
 # From jekyll/jekyll-mentions
 
-require 'rubygems'
-require 'bundler'
+require "rubygems"
+require "bundler"
 
 begin
   Bundler.setup(:default, :development, :test)
@@ -11,25 +11,22 @@ rescue Bundler::BundlerError => e
   exit e.status_code
 end
 
-
 # Test task
 
-require 'rake'
-require 'rake/testtask'
+require "rake"
+require "rake/testtask"
 
 Rake::TestTask.new(:test) do |test|
-  test.libs << 'lib' << 'test'
-  test.pattern = 'test/**/test_*.rb'
-  test.verbose = true
+  test.libs << "lib" << "test"
+  test.pattern = "test/**/test_*.rb"
 end
 
-task :default => 'test'
-
+task :default => "test"
 
 # Release task
 
 def name
-  @name ||= File.basename(Dir['*.gemspec'].first, ".*")
+  @name ||= File.basename(Dir["*.gemspec"].first, ".*")
 end
 
 def version
@@ -46,7 +43,7 @@ end
 
 desc "Release #{name} v#{version}"
 task :release => :build do
-  unless `git branch` =~ /^\* master$/
+  unless `git branch` =~ %r!^\* master$!
     puts "You must be on the master branch to release!"
     exit!
   end

jekyll-archives.gemspec

@@ -15,9 +15,9 @@ Gem::Specification.new do |s|
 
   s.add_dependency "jekyll", ">= 2.4"
 
+  s.add_development_dependency  "minitest"
   s.add_development_dependency  "rake"
   s.add_development_dependency  "rdoc"
-  s.add_development_dependency  "shoulda"
-  s.add_development_dependency  "minitest"
   s.add_development_dependency  "rubocop"
+  s.add_development_dependency  "shoulda"
 end

lib/jekyll-archives/archive.rb

@@ -1,7 +1,6 @@
 module Jekyll
   module Archives
     class Archive < Jekyll::Page
-
       attr_accessor :posts, :type, :slug
 
       # Attributes for Liquid templates
@@ -30,10 +29,9 @@ def initialize(site, title, type, posts)
         @title  = title
         @config = site.config["jekyll-archives"]
 
-        # Generate slug if tag or category (taken from jekyll/jekyll/features/support/env.rb)
-        if title.is_a? String
-          @slug = Utils.slugify(title)
-        end
+        # Generate slug if tag or category
+        # (taken from jekyll/jekyll/features/support/env.rb)
+        @slug = Utils.slugify(title) if title.is_a? String
 
         # Use ".html" for file extension and url for path
         @ext  = File.extname(relative_path)
@@ -68,7 +66,7 @@ def layout
       # desired placeholder replacements. For details see "url.rb".
       def url_placeholders
         if @title.is_a? Hash
-          @title.merge({ :type => @type })
+          @title.merge(:type => @type)
         else
           { :name => @slug, :type => @type }
         end
@@ -88,7 +86,7 @@ def url
       end
 
       def permalink
-        data && data.is_a?(Hash) && data['permalink']
+        data && data.is_a?(Hash) && data["permalink"]
       end
 
       # Add any necessary layouts to this post
@@ -115,7 +113,7 @@ def add_dependencies
           end
         end
       end
-      
+
       # Convert this Convertible's data to a Hash suitable for use by Liquid.
       #
       # Returns the Hash representation of this Convertible.
@@ -132,9 +130,7 @@ def to_liquid(attrs = nil)
       # Returns a String (for tag and category archives) and nil for
       # date-based archives.
       def title
-        if @title.is_a? String
-          @title
-        end
+        @title if @title.is_a? String
       end
 
       # Produce a date object if a date-based archive
@@ -151,8 +147,8 @@ def date
       #
       # Returns the destination relative path String.
       def relative_path
-        path = URL.unescape_path(url).gsub(/^\//, "")
-        path = File.join(path, "index.html") if url =~ /\/$/
+        path = URL.unescape_path(url).gsub(%r!^\/!, "")
+        path = File.join(path, "index.html") if url =~ %r!\/$!
         path
       end
 

test/test_jekyll_archives.rb

@@ -3,10 +3,8 @@
 class TestJekyllArchives < Minitest::Test
   context "the jekyll-archives plugin" do
     setup do
-      @site = fixture_site({
-        "jekyll-archives" => {
-          "enabled" => true
-        }
+      @site = fixture_site("jekyll-archives" => {
+        "enabled" => true
       })
       @site.read
       @archives = Jekyll::Archives::Archives.new(@site.config)
@@ -50,11 +48,9 @@ class TestJekyllArchives < Minitest::Test
 
   context "the jekyll-archives plugin with custom layout path" do
     setup do
-      @site = fixture_site({
-        "jekyll-archives" => {
-          "layout"  => "archive-too",
-          "enabled" => true
-        }
+      @site = fixture_site("jekyll-archives" => {
+        "layout"  => "archive-too",
+        "enabled" => true
       })
       @site.process
     end
@@ -111,10 +107,8 @@ class TestJekyllArchives < Minitest::Test
 
   context "the archives" do
     setup do
-      @site = fixture_site({
-        "jekyll-archives" => {
-          "enabled" => true
-        }
+      @site = fixture_site("jekyll-archives" => {
+        "enabled" => true
       })
       @site.process
     end
@@ -137,10 +131,8 @@ class TestJekyllArchives < Minitest::Test
 
   context "the jekyll-archives plugin with enabled array" do
     setup do
-      @site = fixture_site({
-        "jekyll-archives" => {
-          "enabled" => ["tags"]
-        }
+      @site = fixture_site("jekyll-archives" => {
+        "enabled" => ["tags"]
       })
       @site.process
     end
@@ -161,10 +153,8 @@ class TestJekyllArchives < Minitest::Test
 
   context "the jekyll-archives plugin" do
     setup do
-      @site = fixture_site({
-        "jekyll-archives" => {
-          "enabled" => true
-        }
+      @site = fixture_site("jekyll-archives" => {
+        "enabled" => true
       })
       @site.process
       @archives = @site.config["archives"]