Skip to content

ci: auto-comment the pr-XXX image tag on PRs and linked issues - #5013

Merged
amir20 merged 1 commit into
masterfrom
claude/dozzle-auto-pr-comments-ueb4yj
Sep 5, 2026
Merged

ci: auto-comment the pr-XXX image tag on PRs and linked issues#5013
amir20 merged 1 commit into
masterfrom
claude/dozzle-auto-pr-comments-ueb4yj

Conversation

@amir20

@amir20 amir20 commented Sep 5, 2026

Copy link
Copy Markdown
Owner

After the per-PR image is pushed, post a sticky comment on the PR with
the docker run command, and a one-time comment on each issue the PR
closes so reporters can test without waiting for a release.

Co-Authored-By: Claude Opus 5 noreply@anthropic.com
Claude-Session: https://claude.ai/code/session_016m2chBwXvH2ud1GFpejRh8

After the per-PR image is pushed, post a sticky comment on the PR with
the docker run command, and a one-time comment on each issue the PR
closes so reporters can test without waiting for a release.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_016m2chBwXvH2ud1GFpejRh8
@claude

claude Bot commented Sep 5, 2026

Copy link
Copy Markdown

Reviewed .github/workflows/dev.yml. Overall solid: env-var interpolation (no shell injection), idempotent via HTML markers, and fork PRs are safely excluded transitively through needs: buildx.

A couple of things worth checking:

  • The closing-keyword regex (grep -oiE '(close[sd]?|fix(e[sd])?|resolve[sd]?)...#[0-9]+') only matches shorthand #123. A PR body using Fixes https://github.com/amir20/dozzle/issues/123 won't get picked up, so no issue comment in that case.
  • The sticky PR comment (pr_marker) is posted/updated on every push regardless of draft status; only the linked-issue comments are gated by IS_DRAFT. Probably fine, just confirm that's intended (build/push already happens for drafts today, so no new cost here).
  • if [ "$issue" = "$PR" ] is dead code since a PR and an issue in the same repo can't share a number — harmless, could drop.

@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Try this PR without waiting for a release:

docker run -v /var/run/docker.sock:/var/run/docker.sock -p 8080:8080 amir20/dozzle:pr-5013

Also on ghcr.io/amir20/dozzle:pr-5013 and as amir20/dozzle:pr-5013-alpine. Rebuilt on every push, currently caabcc6.

@amir20
amir20 merged commit 2e8ff49 into master Sep 5, 2026
12 checks passed
@amir20
amir20 deleted the claude/dozzle-auto-pr-comments-ueb4yj branch September 5, 2026 21:35
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