Skip to content

[CI] Migrate to npm-managed local Hugo and standardise Makefile targets#210

Open
hebypaul wants to merge 3 commits into
layer5io:masterfrom
hebypaul:feature/hebypaul/208
Open

[CI] Migrate to npm-managed local Hugo and standardise Makefile targets#210
hebypaul wants to merge 3 commits into
layer5io:masterfrom
hebypaul:feature/hebypaul/208

Conversation

@hebypaul

Copy link
Copy Markdown
Contributor

Notes for Reviewers
This PR updates the local development environment to use an npm-managed version of Hugo (extended) and standardizes the Makefile targets. This streamlines the setup process by removing the need for contributors to globally install Hugo on their local machines, ensuring version consistency between local environments and CI.

Key Changes:

  • Makefile: Added a check-deps target to validate dependencies and rerouted standard commands (site, serve, build, clean) to trigger their respective local npm scripts.
  • package.json: Added dev:* scripts (e.g., dev:build, dev:site) to explicitly handle Hugo commands using the local node module.
  • Documentation: Updated README.md and CONTRIBUTING.md to remove references to a global Hugo installation and emphasize the unified make setup -> make site workflow.

This PR fixes #208

Signed commits

  • Yes, I signed my commits.

Signed-off-by: Heby T Paul <hebytpaul1111@gmail.com>
@github-actions

github-actions Bot commented Jun 11, 2026

Copy link
Copy Markdown

🚀 Preview deployment: https://layer5io.github.io/academy-theme/pr-preview/pr-210/

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

Copy link
Copy Markdown
Contributor

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 updates the local development workflow to use an NPM-managed local Hugo binary instead of requiring a global Hugo installation. It updates the documentation, introduces a dependency check in the Makefile, and adds helper scripts to package.json. The review feedback suggests removing the redundant dependency check from the clean target, reverting the removal of npx for npm-check-updates to handle missing optional dependencies, and simplifying a repetitive sentence in the contribution guide.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread Makefile Outdated
Comment thread package.json Outdated
Comment thread CONTRIBUTING.md Outdated
hebypaul added 2 commits June 11, 2026 11:49
Signed-off-by: Heby T Paul <hebytpaul1111@gmail.com>
Signed-off-by: Heby T Paul <hebytpaul1111@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] Migrate to npm-managed local Hugo and standardise Makefile targets

1 participant