[CI] Migrate to npm-managed local Hugo and standardise Makefile targets#210
[CI] Migrate to npm-managed local Hugo and standardise Makefile targets#210hebypaul wants to merge 3 commits into
Conversation
Signed-off-by: Heby T Paul <hebytpaul1111@gmail.com>
|
🚀 Preview deployment: https://layer5io.github.io/academy-theme/pr-preview/pr-210/ |
There was a problem hiding this comment.
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.
Signed-off-by: Heby T Paul <hebytpaul1111@gmail.com>
Signed-off-by: Heby T Paul <hebytpaul1111@gmail.com>
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:
check-depstarget to validate dependencies and rerouted standard commands (site,serve,build,clean) to trigger their respective local npm scripts.dev:*scripts (e.g.,dev:build,dev:site) to explicitly handle Hugo commands using the local node module.README.mdandCONTRIBUTING.mdto remove references to a global Hugo installation and emphasize the unifiedmake setup->make siteworkflow.This PR fixes #208
Signed commits