Skip to content

fix(ci): normalize repository name to lowercase in docker-publish workflow - #19

Open
Nitro-Builds-Yash wants to merge 1 commit into
arcahyadi:mainfrom
Nitro-Builds-Yash:fix/docker-publish-lowercase-repo-name
Open

Nitro-Builds-Yash wants to merge 1 commit into
arcahyadi:mainfrom
Nitro-Builds-Yash:fix/docker-publish-lowercase-repo-name

Conversation

@Nitro-Builds-Yash

@Nitro-Builds-Yash Nitro-Builds-Yash commented Oct 9, 2026 •

Copy link
Copy Markdown

Summary

Docker / OCI specification and GitHub Container Registry (GHCR) reference formats strictly require image repository names to be lowercase. When users or organizations with uppercase letters in their GitHub usernames (such as Nitro-Builds-Yash) fork this project and push commits, the ci / docker publish workflow crashes with:
ERROR: failed to parse ref "ghcr.io/Nitro-Builds-Yash/odysseus": invalid reference format: repository name must be lowercase.

This pull request normalizes IMAGE_NAME to lowercase using tr '[:upper:]' '[:lower:]' before invocation so that multi-arch image builds and GHCR publishing succeed across all forks.

Target branch

  • This PR targets main (note: upstream repository arcahyadi/odysseus currently maintains main as its default branch).

Linked Issue

Part of #18

Type of Change

  • CI / tooling / configuration
  • Bug fix (non-breaking — fixes a confirmed issue)

Checklist

  • I searched open issues and open PRs — this is not a duplicate.
  • My changes are limited to the scope described above — no unrelated refactors or whitespace changes mixed in.
  • I actually ran the app and verified the change works end-to-end. Type-checks and unit tests are not enough.

How to Test

  1. Fork this repository into a GitHub account or organization containing uppercase letters.
  2. Trigger the ci / docker publish workflow on a branch or push.
  3. Observe that without this change, the docker/build-push-action immediately fails with invalid reference format: repository name must be lowercase.
  4. With this change applied, verify that the Normalize repository name to lowercase step executes and the multi-arch digest build and manifest creation succeed smoothly.

rennf93 pushed a commit to rennf93/odysseus that referenced this pull request Oct 10, 2026
…le-cascade

refactor(tests): read the whole cascade instead of style.css alone
rennf93 pushed a commit to rennf93/odysseus that referenced this pull request Oct 10, 2026
Resolve the overlap with arcahyadi#19 by preserving the whole-cascade stylesheet
helpers alongside #31's document-source/module-set helpers.

Maintainer validation:
- document/module contract and composition guards: 15 passed
- all 54 touched Python test modules: 366 passed
- py_compile: clean
- git diff --check: clean
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