Add rubocop to script/cibuild

.rspec

@@ -0,0 +1,3 @@
+--color
+--require spec_helper
+--order random

.rubocop.yml

@@ -5,3 +5,7 @@ Metrics/LineLength:
   Exclude:
     - spec/**/*
     - jekyll-mentions.gemspec
+
+Metrics/BlockLength:
+  Exclude:
+    - spec/**/*

jekyll-mentions.gemspec

@@ -13,7 +13,6 @@ Gem::Specification.new do |s|
   s.add_dependency "html-pipeline", "~> 2.3"
   s.add_dependency "activesupport", "~> 4.0"
 
-  s.add_development_dependency "rake"
-  s.add_development_dependency "rdoc"
   s.add_development_dependency "rspec", "~> 3.0"
+  s.add_development_dependency "rubocop", "~> 0.48"
 end

lib/jekyll-mentions.rb

@@ -22,6 +22,7 @@ def mentionify(doc)
           doc.output = filter_with_mention(src).call(doc.output)[:output].to_s
         end
       end
+      # rubocop:enable Metrics/AbcSize
 
       # Public: Create or fetch the filter for the given {{src}} base URL.
       #
@@ -30,7 +31,7 @@ def mentionify(doc)
       # Returns an HTML::Pipeline instance for the given base URL.
       def filter_with_mention(src)
         filters[src] ||= HTML::Pipeline.new([
-          HTML::Pipeline::MentionFilter
+          HTML::Pipeline::MentionFilter,
         ], { :base_url => src, :username_pattern => mention_username_pattern })
       end
 
@@ -97,6 +98,6 @@ def default_mention_base
   end
 end
 
-Jekyll::Hooks.register [:pages, :documents], :post_render do |doc|
+Jekyll::Hooks.register %i[pages documents], :post_render do |doc|
   Jekyll::Mentions.mentionify(doc) if Jekyll::Mentions.mentionable?(doc)
 end

script/cibuild

@@ -1,3 +1,5 @@
 #! /bin/sh
 
-bundle exec rspec --color $@
+bundle exec rspec
+bundle exec rubocop -S -D
+gem build jekyll-mentions.gemspec

spec/mentions_spec.rb

@@ -1,5 +1,3 @@
-require "spec_helper"
-
 RSpec.describe(Jekyll::Mentions) do
   Jekyll.logger.log_level = :error
 
@@ -9,7 +7,7 @@
       "skip_config_files" => false,
       "collections"       => { "docs" => { "output" => true } },
       "source"            => fixtures_dir,
-      "destination"       => fixtures_dir("_site")
+      "destination"       => fixtures_dir("_site"),
     }))
   end
   let(:mentions)    { described_class }
@@ -83,7 +81,7 @@ def para(content)
     let(:mentions_src) { "http://mine.club" }
     let(:config_overrides) do
       {
-        "jekyll-mentions" => { "base_url" => mentions_src }
+        "jekyll-mentions" => { "base_url" => mentions_src },
       }
     end
 

spec/spec_helper.rb

@@ -7,7 +7,7 @@ def fixtures_dir(*paths)
   end
 
   def find_by_title(docs, title)
-    docs.find { |d| d.title == title }
+    docs.find { |d| d.data["title"] == title }
   end
 
   # rspec-expectations config goes here. You can use an alternate