Switch Docsy theme to Hugo Modules and unblock Hugo 0.161 upgrade#99
Open
brooke-hamilton wants to merge 2 commits into
Open
Switch Docsy theme to Hugo Modules and unblock Hugo 0.161 upgrade#99brooke-hamilton wants to merge 2 commits into
brooke-hamilton wants to merge 2 commits into
Conversation
Bumps the all group with 2 updates in the /radblog directory: [autoprefixer](https://github.com/postcss/autoprefixer) and [hugo-extended](https://github.com/jakejarvis/hugo-extended). Updates `autoprefixer` from 10.4.27 to 10.5.0 - [Release notes](https://github.com/postcss/autoprefixer/releases) - [Changelog](https://github.com/postcss/autoprefixer/blob/main/CHANGELOG.md) - [Commits](postcss/autoprefixer@10.4.27...10.5.0) Updates `hugo-extended` from 0.102.3 to 0.161.1 - [Commits](jakejarvis/hugo-extended@v0.102.3...v0.161.1) --- updated-dependencies: - dependency-name: autoprefixer dependency-version: 10.5.0 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: all - dependency-name: hugo-extended dependency-version: 0.161.1 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: all ... Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: Brooke Hamilton <45323234+brooke-hamilton@users.noreply.github.com>
Static Web App Preview
Deployed from commit |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Replaces the vendored copy of the Docsy theme with a Hugo Modules import, and brings the Hugo CLI bump from #98 along for the ride.
This unblocks the failing CI build on #98. Once this PR merges, #98 can be closed (its
hugo-extendedbump is included here as the first commit on this branch).Why CI was failing on #98
PR #98 bumps
hugo-extendedfrom^0.102.3to^0.161.1. Hugo 0.161 removed the internal template_internal/google_analytics_async.html(deprecated in v0.123 — it was the snippet for legacyUA-...Analytics IDs; only theG-...GA4 variant remains). The vendored Docsy inradblog/themes/docsy/layouts/partials/head.htmlstill referenced that template in itselsebranch, so the build errored:Patching the vendored file would work, but the underlying problem is that Docsy was checked in at repo bootstrap and never updated since (Docsy
v0.6.0→ currentv0.15.0). A stale.gitmodulesentry pointed at a non-existentdocs/themes/docsysubmodule, andradblog/go.modhad a danglinggithub.com/google/docsy v0.6.0 // indirectrequirement — suggesting a half-finished Hugo Modules conversion that this PR completes.Changes
radblog/config.toml— removedtheme = "docsy"; added[module].importsforgithub.com/google/docsyandgithub.com/google/docsy/dependencies.radblog/go.mod/radblog/go.sum—hugo mod get -uupgraded Docsyv0.6.0→v0.15.0and addeddocsy/dependencies v0.7.2.radblog/themes/docsy/tree (~260 files); Hugo now pulls Docsy from the module cache..gitmodules(the submodule it declared has never existed in this repo)..github/workflows/blog.yaml— dropped the now-brokenInstall Docsy dependenciesstep (its working directory no longer exists) and thesubmodules: truecheckout flag (no submodules in the repo).hugo-extended^0.102.3→^0.161.1inradblog/package.jsonandradblog/package-lock.json.Dependabot already has a
gomodecosystem configured for/radblogin .github/dependabot.yml, so future Docsy releases will be picked up automatically.Validation
Built locally with Hugo 0.161.1 extended:
Verified GA4 still renders in
public/index.html:Two pre-existing deprecation warnings remain (out of scope, non-blocking):
languageCodeconfig key is deprecated in favor oflocale..Language.LanguageDirectionused inside Docsy itself; will be fixed by a future Docsy release.Follow-up