Skip to content

Document submodule init step for Docker workflow#132

Closed
yqtian-se wants to merge 2 commits intoacmsigsoft:mainfrom
yqtian-se:readme-docker-submodules
Closed

Document submodule init step for Docker workflow#132
yqtian-se wants to merge 2 commits intoacmsigsoft:mainfrom
yqtian-se:readme-docker-submodules

Conversation

@yqtian-se
Copy link
Copy Markdown
Contributor

Without initializing the hugo-universal-theme submodule, the Docker build only produces a handful of pages and most URLs return 404.

Copy link
Copy Markdown
Contributor

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 updates the local development documentation to make the Docker workflow produce a complete Hugo site build. It fits into the repository by clarifying setup for contributors working on the SIGSOFT website without a local Hugo installation.

Changes:

  • Add a git submodule update --init --recursive step to the Docker-based local preview/build instructions.
  • Explain that the Hugo theme submodule must be present or the generated site will be incomplete and many routes will 404.

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

yqtian-se added 2 commits May 2, 2026 12:13
The dart-sass-embedded snap is deprecated and snap store outages
were causing transient failures ("unable to contact snap store").
Pulling the prebuilt binary from the sass/dart-sass GitHub release
is what Hugo's official docs now recommend.
Without initializing the hugo-universal-theme submodule, the Docker
build only produces a handful of pages and most URLs return 404.
@yqtian-se yqtian-se force-pushed the readme-docker-submodules branch from ecfbf75 to ad217ea Compare May 2, 2026 02:15
@yqtian-se yqtian-se closed this May 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants