Skip to content

Initial image set up - #4

Merged
byan1197 merged 3 commits into
mainfrom
setup-ci
Oct 16, 2026
Merged

byan1197 merged 3 commits into
mainfrom
setup-ci

Conversation

@byan1197

@byan1197 byan1197 commented Oct 9, 2026

Copy link
Copy Markdown
Collaborator
  • Bundles python dependencies with Zensical
  • Sets up CI
  • Added codeowners @backstage/techdocs-maintainers
  • Added mock docs for testing

setting up ci

Signed-off-by: Bond Yan <bondy@spotify.com>
Signed-off-by: Bond Yan <bondy@spotify.com>
@byan1197
byan1197 requested a review from a team as a code owner October 9, 2026 21:00
Signed-off-by: Bond Yan <bondy@spotify.com>

@melissawimberly melissawimberly 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.

Two things to look at, but otherwise good to go

  1. latest gets overwritten by unreleased code (.github/workflows/release.yml:29): every push to main builds and pushes ghcr.io//techdocs-zensical:latest, and a published release also tags latest. So any merge after a release replaces the released image with unreleased code. A fix would be to tag main builds as something like edge or main, or the commit SHA, and keep latest for releases only.

  2. The build may fail on the new base image (Dockerfile:17-21): the base changed from python:3.14-alpine to zensical/zensical:0.0.69, but the next step still runs apk update && apk add ... graphviz plantuml. If the new image isn't Alpine-based, apk won't exist and the build breaks there. Running docker run --rm zensical/zensical:0.0.69 cat /etc/os-release would settle it.

@byan1197
byan1197 merged commit 44f0f76 into main Oct 16, 2026
2 checks passed
@byan1197
byan1197 deleted the setup-ci branch October 16, 2026 15:31
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