Skip to content

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

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

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

Conversation

@hebypaul

@hebypaul hebypaul commented Jun 6, 2026

Copy link
Copy Markdown
Contributor

Notes for Reviewers

This PR migrates the exoscale-academy build process to use an npm-managed local Hugo dependency, standardizing the local development experience and mitigating version mismatch issues.

Specific updates include:

  • Adding a check-deps target to the Makefile to validate npm and the local hugo binary.
  • Rerouting site, serve, build, and clean commands to trigger npm run scripts.
  • Adding dev:* scripts to package.json to handle Hugo commands with the correct flags, stripping out unnecessary npx prefixes.
  • Updating README.md to remove references to a globally installed Hugo and emphasize the new make setup -> make site workflow.

This PR fixes #407

Signed commits

  • Yes, I signed my commits.

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

@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 migrates the project's build and development workflow to use a locally managed Hugo binary via npm, updating the Makefile, README, and package.json scripts accordingly. Feedback on these changes highlights that the theme-update target in the Makefile still incorrectly references the global hugo binary instead of the local one. Additionally, the clean target in the Makefile and the dev:clean script in package.json are redundant and inefficient, and should be refactored or removed to properly clean build artifacts.

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
Comment thread Makefile
Comment thread package.json Outdated
@github-actions

github-actions Bot commented Jun 6, 2026

Copy link
Copy Markdown
Contributor

🚀 Preview deployment: https://layer5io.github.io/exoscale-academy/pr-preview/pr-408/

Note: Preview may take a moment (GitHub Pages deployment in progress). Please wait and refresh. Track deployment here

hebypaul added 2 commits June 7, 2026 14:01
Signed-off-by: Heby T Paul <hebytpaul1111@gmail.com>
… update scripts

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