Inherit Jekyll's rubocop config for consistency

.rubocop.yml

@@ -0,0 +1,7 @@
+inherit_gem:
+  jekyll: .rubocop.yml
+
+Metrics/LineLength:
+  Exclude:
+    - spec/**/*
+    - jekyll-mentions.gemspec

Rakefile

@@ -1,18 +1,18 @@
-require 'rubygems'
-require 'bundler'
+require "rubygems"
+require "bundler"
 begin
   Bundler.setup(:default, :development, :test)
 rescue Bundler::BundlerError => e
   $stderr.puts e.message
   $stderr.puts "Run `bundle install` to install missing gems"
   exit e.status_code
 end
-require 'bundler/gem_tasks'
-require 'rake'
-require 'rake/testtask'
+require "bundler/gem_tasks"
+require "rake"
+require "rake/testtask"
 
 Rake::TestTask.new(:test) do |test|
-  test.libs << 'lib' << 'test'
-  test.pattern = 'test/**/test_*.rb'
+  test.libs << "lib" << "test"
+  test.pattern = "test/**/test_*.rb"
   test.verbose = true
 end

jekyll-mentions.gemspec

@@ -9,10 +9,11 @@ Gem::Specification.new do |s|
   s.licenses    = ["MIT"]
   s.files       = [ "lib/jekyll-mentions.rb" ]
 
-  s.add_dependency "jekyll", '~> 3.0'
-  s.add_dependency "html-pipeline", '~> 2.3'
+  s.add_dependency "jekyll", "~> 3.0"
+  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 "rake"
+  s.add_development_dependency "rdoc"
+  s.add_development_dependency "rspec", "~> 3.0"
 end

lib/jekyll-mentions.rb

@@ -1,21 +1,21 @@
-require 'jekyll'
-require 'html/pipeline'
+require "jekyll"
+require "html/pipeline"
 
 module Jekyll
   class Mentions
     GITHUB_DOT_COM = "https://github.com".freeze
     BODY_START_TAG = "<body".freeze
 
-
     InvalidJekyllMentionConfig = Class.new(Jekyll::Errors::FatalException)
 
     class << self
+      # rubocop:disable Metrics/AbcSize
       def mentionify(doc)
         return unless doc.output.include?("@")
         src = mention_base(doc.site.config)
         if doc.output.include? BODY_START_TAG
           parsed_doc    = Nokogiri::HTML::Document.parse(doc.output)
-          body          = parsed_doc.at_css('body')
+          body          = parsed_doc.at_css("body")
           body.children = filter_with_mention(src).call(body.inner_html)[:output].to_s
           doc.output    = parsed_doc.to_html
         else
@@ -31,11 +31,14 @@ def mentionify(doc)
       def filter_with_mention(src)
         filters[src] ||= HTML::Pipeline.new([
           HTML::Pipeline::MentionFilter
-        ], { :base_url => src , :username_pattern => mention_username_pattern })
+        ], { :base_url => src, :username_pattern => mention_username_pattern })
       end
 
       def mention_username_pattern
-        Regexp.new(HTML::Pipeline::MentionFilter::UsernamePattern.source, Regexp::IGNORECASE)
+        Regexp.new(
+          HTML::Pipeline::MentionFilter::UsernamePattern.source,
+          Regexp::IGNORECASE
+        )
       end
 
       # Public: Filters hash where the key is the mention base URL.
@@ -55,14 +58,14 @@ def filters
       # Returns a URL to use as the base URL for mentions.
       # Defaults to the https://github.com.
       def mention_base(config = {})
-        mention_config = config['jekyll-mentions']
+        mention_config = config["jekyll-mentions"]
         case mention_config
         when nil, NilClass
           GITHUB_DOT_COM
         when String
           mention_config.to_s
         when Hash
-          mention_config.fetch('base_url', GITHUB_DOT_COM)
+          mention_config.fetch("base_url", GITHUB_DOT_COM)
         else
           raise InvalidJekyllMentionConfig,
             "Your jekyll-mentions config has to either be a" \

spec/mentions_spec.rb

@@ -1,15 +1,15 @@
-require 'spec_helper'
+require "spec_helper"
 
 RSpec.describe(Jekyll::Mentions) do
   Jekyll.logger.log_level = :error
 
   let(:config_overrides) { {} }
   let(:configs) do
     Jekyll.configuration(config_overrides.merge({
-      'skip_config_files' => false,
-      'collections'       => { 'docs' => { 'output' => true } },
-      'source'            => fixtures_dir,
-      'destination'       => fixtures_dir('_site')
+      "skip_config_files" => false,
+      "collections"       => { "docs" => { "output" => true } },
+      "source"            => fixtures_dir,
+      "destination"       => fixtures_dir("_site")
     }))
   end
   let(:mentions)    { described_class }

spec/spec_helper.rb

@@ -1,7 +1,7 @@
-require File.expand_path('../../lib/jekyll-mentions.rb', __FILE__)
+require File.expand_path("../../lib/jekyll-mentions.rb", __FILE__)
 
 RSpec.configure do |config|
-  FIXTURES_DIR = File.expand_path('../fixtures', __FILE__)
+  FIXTURES_DIR = File.expand_path("../fixtures", __FILE__)
   def fixtures_dir(*paths)
     File.join(FIXTURES_DIR, *paths)
   end