Skip to content

chore(ci): update hugo starter to use mise defined binary - #194

Merged
coryrylan merged 1 commit into
mainfrom
topic-hugo-bin
Jul 28, 2026
Merged

chore(ci): update hugo starter to use mise defined binary#194
coryrylan merged 1 commit into
mainfrom
topic-hugo-bin

Conversation

@coryrylan

@coryrylan coryrylan commented Jul 28, 2026

Copy link
Copy Markdown
Collaborator
  • Updated mise.toml to include Hugo as a required tool.
  • Added Hugo installation details to the BUILD documentation.
  • Removed outdated hugo-bin dependency from pnpm-lock.yaml and pnpm-workspace.yaml.
  • Updated CI validation script to include Hugo in the required tools list.
  • Adjusted package.json in the Hugo starter to reflect the new installation process.

Summary by CodeRabbit

  • New Features

    • Hugo starter projects now use a consistent, repository-managed Hugo version for builds and local development.
    • Hugo builds automatically respond to changes in the shared tool configuration.
  • Bug Fixes

    • Starter project exports now correctly handle wrapped build commands and scripts without command definitions.
  • Documentation

    • Updated Hugo setup instructions to explain installing the pinned version through the repository tooling and link to official installation guidance.

- Updated mise.toml to include Hugo as a required tool.
- Added Hugo installation details to the BUILD documentation.
- Removed outdated hugo-bin dependency from pnpm-lock.yaml and pnpm-workspace.yaml.
- Updated CI validation script to include Hugo in the required tools list.
- Adjusted package.json in the Hugo starter to reflect the new installation process.

Signed-off-by: Cory Rylan <crylan@nvidia.com>
@coryrylan coryrylan self-assigned this Jul 28, 2026
Copilot AI review requested due to automatic review settings July 28, 2026 15:58

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions github-actions Bot added scope(ci) scope(internals) dependencies Pull requests that update a dependency file scope(starters) labels Jul 28, 2026
@coderabbitai

coderabbitai Bot commented Jul 28, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Enterprise

Run ID: c62c1e68-476d-489f-ac83-44570f10d0bf

📥 Commits

Reviewing files that changed from the base of the PR and between 0a6b54e and c68ef71.

⛔ Files ignored due to path filters (2)
  • mise.lock is excluded by !**/*.lock
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (8)
  • mise.toml
  • pnpm-workspace.yaml
  • projects/internals/BUILD.md
  • projects/internals/ci/ci-validate.js
  • projects/internals/tools/src/project/starters.test.ts
  • projects/internals/tools/src/project/starters.ts
  • projects/starters/hugo/README.md
  • projects/starters/hugo/package.json
💤 Files with no reviewable changes (1)
  • pnpm-workspace.yaml

📝 Walkthrough

Walkthrough

Hugo is added to the repository’s mise toolchain and CI validation. The Hugo starter invokes Hugo through mise, watches mise.toml, updates installation guidance, removes hugo-bin, and adjusts package export handling for mise-wrapped or missing Wireit commands.

Changes

Hugo mise integration

Layer / File(s) Summary
mise toolchain and validation
mise.toml, pnpm-workspace.yaml, projects/internals/BUILD.md, projects/internals/ci/ci-validate.js
Hugo 0.152.2 is added to the mise toolchain, hugo-bin is removed from build permissions, documentation lists Hugo, and CI validates the required tool.
Hugo starter mise commands
projects/starters/hugo/package.json, projects/starters/hugo/README.md
Build and development commands run through mise exec; builds watch mise.toml; and setup documentation describes installing the pinned Hugo version.
Package export normalization
projects/internals/tools/src/project/starters.ts, projects/internals/tools/src/project/starters.test.ts
Exported package scripts strip mise exec --, omit scripts without Wireit commands, remove the Wireit metadata, and test both behaviors.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Suggested labels: scope(docs)

Suggested reviewers: copilot

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title matches the main change: switching the Hugo starter to a mise-defined Hugo binary.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch topic-hugo-bin

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

projects/internals/tools/src/project/starters.test.ts

Parsing error: error TS5012: Cannot read file '/tsconfig.json': ENOENT: no such file or directory, open '/tsconfig.json'.


Comment @coderabbitai help to get the list of available commands.

@coryrylan
coryrylan merged commit 29211a7 into main Jul 28, 2026
15 checks passed
@coryrylan
coryrylan deleted the topic-hugo-bin branch July 28, 2026 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file scope(ci) scope(internals) scope(starters)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants