Skip to content

Port site to Hugo#60

Open
jimmac wants to merge 1 commit into
mainfrom
wip/jimmac/hugo-port
Open

Port site to Hugo#60
jimmac wants to merge 1 commit into
mainfrom
wip/jimmac/hugo-port

Conversation

@jimmac
Copy link
Copy Markdown
Collaborator

@jimmac jimmac commented May 11, 2026

  • Jekyll is a dependency hell
  • Zola would be my choice, but since Toolbx is written in Go, it makes more sense to stick with Hugo. Majority of data is markdown anyway.

@jimmac jimmac force-pushed the wip/jimmac/hugo-port branch from a44e7f9 to 6ea3a1a Compare May 11, 2026 10:14
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request migrates the Toolbx website from Jekyll to Hugo, replacing the previous build system and directory structure. The changes include updating the .gitignore, rewriting the README instructions, and converting Jekyll layouts and includes into Hugo templates and partials. Review feedback identifies a bug in accessing data files, suggests more idiomatic usage of Hugo's relURL function for assets and links, and highlights a maintainability regression where version numbers were hardcoded instead of using configurable parameters.

Comment thread layouts/partials/footer.html Outdated
Comment thread layouts/partials/footer.html Outdated
Comment thread layouts/partials/head.html Outdated
Comment thread content/distros.md Outdated
@jimmac jimmac force-pushed the wip/jimmac/hugo-port branch from 6ea3a1a to ebe0785 Compare May 11, 2026 10:20
- Jekyll is a dependency hell
- Zola would be my choice, but since Toolbx is written in Go, it makes
more sense to stick with Hugo. Majority of data is markdown anyway.

Signed-off-by: Jakub Steiner <jimmac@gmail.com>
@jimmac jimmac force-pushed the wip/jimmac/hugo-port branch from ebe0785 to a3776f8 Compare May 11, 2026 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant