Bump rubocop-jekyll to 0.13.0 and fix rubocop issues (#252)

Merge pull request 252

.rubocop.yml

@@ -10,7 +10,7 @@ AllCops:
     - vendor/**/*
     - script/*
 
-Metrics/LineLength:
+Layout/LineLength:
   Enabled: false
 
 Metrics/BlockLength:
@@ -24,6 +24,9 @@ Metrics/MethodLength:
 Style/AccessModifierDeclarations:
   Enabled: false
 
+Style/FetchEnvVar:
+  Enabled: false
+
 Style/FrozenStringLiteralComment:
   Exclude:
     - lib/jekyll-github-metadata/site_github_munger.rb

jekyll-github-metadata.gemspec

@@ -24,5 +24,5 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "pry"
   spec.add_development_dependency "rake"
   spec.add_development_dependency "rspec", "~> 3.12.0"
-  spec.add_development_dependency "rubocop-jekyll", "~> 0.12.0"
+  spec.add_development_dependency "rubocop-jekyll", "~> 0.13.0"
 end

lib/jekyll-github-metadata/client.rb

@@ -130,11 +130,11 @@ def client_inspect
       def pluck_auth_method
         if ENV["JEKYLL_GITHUB_TOKEN"] || Octokit.access_token
           { :access_token => ENV["JEKYLL_GITHUB_TOKEN"] || Octokit.access_token }
-        elsif !ENV["NO_NETRC"] && File.exist?(File.join(ENV["HOME"], ".netrc")) && safe_require("netrc")
+        elsif !ENV["NO_NETRC"] && File.exist?(File.join(Dir.home, ".netrc")) && safe_require("netrc")
           { :netrc => true }
         else
-          GitHubMetadata.log :warn, "No GitHub API authentication could be found." \
-            " Some fields may be missing or have incorrect data."
+          GitHubMetadata.log :warn, "No GitHub API authentication could be found. " \
+                                    "Some fields may be missing or have incorrect data."
           {}.freeze
         end
       end

lib/jekyll-github-metadata/repository_finder.rb

@@ -27,9 +27,9 @@ def nwo
             nwo_from_git_origin_remote || \
             proc do
               raise NoRepositoryError, "No repo name found. " \
-                "Specify using PAGES_REPO_NWO environment variables, " \
-                "'repository' in your configuration, or set up an 'origin' " \
-                "git remote pointing to your github.com repository."
+                                       "Specify using PAGES_REPO_NWO environment variables, " \
+                                       "'repository' in your configuration, or set up an 'origin' " \
+                                       "git remote pointing to your github.com repository."
             end.call
         end
       end

spec/edit_link_tag_spec.rb

@@ -26,7 +26,7 @@
 
   subject do
     tag = described_class.parse(tag_name, markup, tokenizer, parse_context)
-    tag.instance_variable_set("@context", render_context)
+    tag.instance_variable_set(:@context, render_context)
     tag
   end
 

spec/metadata_drop_spec.rb

@@ -86,7 +86,7 @@
 
     context "with fallback data" do
       let(:fallback_data) { { "foo" => "bar" } }
-      before { subject.instance_variable_set("@fallback_data", fallback_data) }
+      before { subject.instance_variable_set(:@fallback_data, fallback_data) }
 
       it "returns the mutated value via #[]" do
         expect(subject["foo"]).to eql("bar")