Gemfile

@@ -9,3 +9,5 @@ gem "jekyll", "~> #{ENV["JEKYLL_VERSION"]}" if ENV["JEKYLL_VERSION"]
 group :test do
   gem "webmock", "~> 3.14"
 end
+
+gem "webrick" if RUBY_VERSION.to_i >= 3

History.markdown

@@ -1,3 +1,13 @@
+## 2.15.0 / 2022-05-20
+
+### Bug Fixes
+
+  * Inject `site.github` via `:pre_render` step rather than `:after_init` (#238)
+
+### Documentation
+
+  * Add GitHub Actions badge and remove Travis one (#236)
+
 ## 2.14.0 / 2022-05-04
 
 ### Minor Enhancements

appveyor.yml

@@ -21,7 +21,6 @@ environment:
       JEKYLL_VERSION:  "3.7"
       FARADAY_VERSION: "0.17"
     - RUBY_FOLDER_VER: "25"
-    - RUBY_FOLDER_VER: "24"
 
 install:
   - SET PATH=C:\Ruby%RUBY_FOLDER_VER%-x64\bin;%PATH%

docs/README.md

@@ -1,6 +1,6 @@
 # GitHub Metadata, a.k.a. `site.github`
 
-[![Linux Build Status](https://img.shields.io/travis/jekyll/github-metadata/master.svg?label=Linux%20build)][travis]
+[![Actions CI](https://github.com/jekyll/github-metadata/actions/workflows/ci.yaml/badge.svg)](https://github.com/jekyll/github-metadata/actions/workflows/ci.yaml)
 [![Windows Build status](https://img.shields.io/appveyor/ci/jekyll/github-metadata/master.svg?label=Windows%20build)][appveyor]
 
 [travis]: https://travis-ci.org/jekyll/github-metadata

lib/jekyll-github-metadata/site_github_munger.rb

@@ -6,6 +6,10 @@ module GitHubMetadata
     class SiteGitHubMunger
       extend Forwardable
 
+      class << self
+        attr_accessor :global_munger
+      end
+
       def_delegators Jekyll::GitHubMetadata, :site, :repository
       private :repository
 
@@ -16,13 +20,14 @@ def initialize(site)
       def munge!
         Jekyll::GitHubMetadata.log :debug, "Initializing..."
 
-        # This is the good stuff.
-        site.config["github"] = github_namespace
-
         add_title_and_description_fallbacks!
         add_url_and_baseurl_fallbacks! if should_add_url_fallbacks?
       end
 
+      def inject_metadata!(payload)
+        payload.site["github"] = github_namespace
+      end
+
       private
 
       def github_namespace
@@ -80,9 +85,14 @@ def should_warn_about_site_name?
         site.config["name"] && !site.config["title"]
       end
     end
-  end
-end
 
-Jekyll::Hooks.register :site, :after_init do |site|
-  Jekyll::GitHubMetadata::SiteGitHubMunger.new(site).munge!
+    Jekyll::Hooks.register :site, :after_init do |site|
+      SiteGitHubMunger.global_munger = SiteGitHubMunger.new(site)
+      SiteGitHubMunger.global_munger.munge!
+    end
+
+    Jekyll::Hooks.register :site, :pre_render do |_site, payload|
+      SiteGitHubMunger.global_munger.inject_metadata!(payload)
+    end
+  end
 end

lib/jekyll-github-metadata/version.rb

@@ -2,6 +2,6 @@
 
 module Jekyll
   module GitHubMetadata
-    VERSION = "2.14.0"
+    VERSION = "2.15.0"
   end
 end

script/test-site

@@ -1,2 +1,2 @@
 #!/bin/bash
-bundle exec jekyll serve -s spec/test-site
+bundle exec jekyll serve -s spec/test-site $@

spec/site_github_munger_spec.rb

@@ -10,44 +10,46 @@
   let(:site) { Jekyll::Site.new(Jekyll::Configuration.from(user_config)) }
   subject { described_class.new(site) }
   let!(:stubs) { stub_all_api_requests }
+  let(:unified_payload) { site.site_payload }
 
   context "generating" do
     before(:each) do
       ENV["JEKYLL_ENV"] = "production"
       subject.munge!
+      subject.inject_metadata!(unified_payload)
     end
 
     context "with site.github as nil" do
-      it "replaces site.github with the drop" do
-        expect(site.config["github"]).to be_a(Liquid::Drop)
+      it "sets site.github to the drop" do
+        expect(unified_payload.site["github"]).to be_a(Liquid::Drop)
       end
     end
 
     context "without site.github" do
       let(:user_config) { {} }
 
       it "replaces site.github with the drop" do
-        expect(site.config["github"]).to be_a(Liquid::Drop)
+        expect(unified_payload.site["github"]).to be_a(Liquid::Drop)
       end
     end
 
     context "with site.github as a non-hash" do
       let(:github_namespace) { "foo" }
 
       it "doesn't munge" do
-        expect(site.config["github"]).to eql("foo")
+        expect(unified_payload.site["github"]).to eql("foo")
       end
     end
 
     context "with site.github as a hash" do
       let(:github_namespace) { { "source" => { "branch" => "foo" } } }
 
       it "lets user-specified values override the drop" do
-        expect(site.config["github"].invoke_drop("source")["branch"]).to eql("foo")
+        expect(unified_payload.site["github"].invoke_drop("source")["branch"]).to eql("foo")
       end
 
       it "still sets other values" do
-        expect(site.config["github"].invoke_drop("source")["path"]).to eql("/")
+        expect(unified_payload.site["github"].invoke_drop("source")["path"]).to eql("/")
       end
     end
 
@@ -202,8 +204,8 @@
     end
 
     it "sets the site.github config" do
-      subject.munge!
-      expect(site.config["github"]).to be_instance_of(Jekyll::GitHubMetadata::MetadataDrop)
+      subject.inject_metadata!(unified_payload)
+      expect(unified_payload.site["github"]).to be_instance_of(Jekyll::GitHubMetadata::MetadataDrop)
     end
   end
 
@@ -223,8 +225,9 @@
 
     it "fails loudly upon call to any drop method" do
       subject.munge!
+      subject.inject_metadata!(unified_payload)
       expect do
-        site.config["github"]["url"]
+        unified_payload.site["github"]["url"]
       end.to raise_error(Jekyll::GitHubMetadata::Client::BadCredentialsError)
     end
   end