Skip to content

fix(ci): use dynamic repository owner for Docker image#2000

Open
roeidalm wants to merge 1 commit intokoala73:mainfrom
roeidalm:fix/docker-publish-dynamic-owner
Open

fix(ci): use dynamic repository owner for Docker image#2000
roeidalm wants to merge 1 commit intokoala73:mainfrom
roeidalm:fix/docker-publish-dynamic-owner

Conversation

@roeidalm
Copy link
Copy Markdown

Summary

  • Replace hardcoded ghcr.io/koala73 with ghcr.io/${{ github.repository_owner }} so forks can publish to their own GHCR namespace without modifying the workflow
  • Enable latest tag on workflow_dispatch in addition to release events

Motivation

Forks currently cannot use the Docker publish workflow as-is because the image name is hardcoded to koala73. Using github.repository_owner makes the workflow portable across forks while maintaining the same behavior for the upstream repo.

The latest tag on workflow_dispatch allows manual builds to produce a usable tag without creating a full release.

Test plan

  • Triggered workflow_dispatch on fork — image pushed to ghcr.io/roeidalm/worldmonitor:latest
  • Verified ${{ github.repository_owner }} resolves correctly

🤖 Generated with Claude Code

… on dispatch

- Replace hardcoded `ghcr.io/koala73` with `ghcr.io/${{ github.repository_owner }}`
  so forks publish to their own GHCR namespace
- Enable `latest` tag on `workflow_dispatch` (was release-only)

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
@vercel
Copy link
Copy Markdown

vercel bot commented Mar 21, 2026

@roeidalm is attempting to deploy a commit to the Elie Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions bot added the trust:caution Brin: contributor trust score caution label Mar 21, 2026
@koala73 koala73 added Low Value Trivial, unnecessary, or not aligned with project needs Ready to Merge PR is mergeable, passes checks, and adds value labels Mar 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Low Value Trivial, unnecessary, or not aligned with project needs Ready to Merge PR is mergeable, passes checks, and adds value trust:caution Brin: contributor trust score caution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants