Skip to content

Conversation

@yankay
Copy link
Member

@yankay yankay commented Dec 3, 2025

What type of PR is this?

/kind cleanup

What this PR does / why we need it

Description

Due to the current Hugo and Docsy versions being too outdated for maintenance.
This PR upgrades the Hugo version for the documentation site to v0.152.2 and migrates the Docsy theme management to use Go modules. Reference: kubernetes-sigs/kueue#1884

Changes

  • Makefile: Updated the hugo target to install v0.152.2 with extended tags. Added site-server target. Updated prepare-release-branch to modify hugo.toml.
  • Netlify: Updated netlify.toml to use Hugo v0.152.2
  • Site Config: Migrated site/config.toml to site/hugo.toml and initialized site/go.mod to manage github.com/google/docsy.
  • Templates:
  • Fixed feedback.html to correctly reference .Site.Params.ui.feedback.
  • Simplified Google Analytics logic in head.html.

Which issue(s) this PR fixes

Fixes #

Special notes for your reviewer

Does this PR introduce a user-facing change?

Upgrade Hugo to v0.152.2 and migrate site to Docsy Go module

Copilot AI review requested due to automatic review settings December 3, 2025 06:54
@k8s-ci-robot k8s-ci-robot added the kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. label Dec 3, 2025
@netlify
Copy link

netlify bot commented Dec 3, 2025

Deploy Preview for kubernetes-sigs-lws canceled.

Name Link
🔨 Latest commit 6a90fdd
🔍 Latest deploy log https://app.netlify.com/projects/kubernetes-sigs-lws/deploys/693271e2ecfc0a0008660a7b

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Dec 3, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR upgrades the Hugo documentation tooling from v0.92.0 to v0.152.2 and migrates from npm-based Docsy theme management to Hugo's Go module system, modernizing the documentation infrastructure.

Key changes:

  • Hugo upgraded from v0.92.0 to v0.152.2 across all environments (Netlify and local development)
  • Docsy theme migrated from npm to Go modules (v0.13.0)
  • Configuration migrated from config.toml to hugo.toml with Hugo module imports

Reviewed changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
Makefile Updated hugo installation to v0.152.2 with extended tags, added site-server target, and updated prepare-release-branch to reference hugo.toml
netlify.toml Updated Hugo version to v0.152.2 across all deployment contexts (production, deploy-preview, branch-deploy)
site/hugo.toml Migrated from config.toml, removed npm theme configuration, added Go module imports for Docsy, and changed disableKinds from ["taxonomy", "taxonomyTerm"] to ["taxonomy"]
site/go.mod Created new Go module file for site with Docsy v0.13.0 dependency
site/go.sum Added checksums for Docsy v0.13.0 and its dependencies (Font-Awesome, Bootstrap)
site/layouts/partials/head.html Simplified Google Analytics template logic to use single internal template
site/layouts/partials/feedback.html Fixed template references to correctly access feedback configuration via .Site.Params.ui.feedback

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@k8s-ci-robot k8s-ci-robot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Dec 3, 2025
@yankay yankay force-pushed the upgrade-hugo2 branch 3 times, most recently from 98b1755 to 3778530 Compare December 3, 2025 07:55
@Edwinhr716
Copy link
Contributor

/lgtm
/approve

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Dec 5, 2025
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Edwinhr716, yankay

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 5, 2025
@k8s-ci-robot k8s-ci-robot merged commit c5e6f17 into kubernetes-sigs:main Dec 5, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants