Skip to content

ci(docker): add workflow_dispatch + retrigger on own workflow edits - #867

Merged
LucasSantana-Dev merged 1 commit into
mainfrom
fix/docker-publish-triggers
May 15, 2026
Merged

LucasSantana-Dev merged 1 commit into
mainfrom
fix/docker-publish-triggers

Conversation

@LucasSantana-Dev

@LucasSantana-Dev LucasSantana-Dev commented May 15, 2026 •

Copy link
Copy Markdown
Owner

Follow-up to #866. The hotfix only touched .github/workflows/docker-publish.yml, but that path was not in the workflow's own push trigger — so the workflow did not re-run and v2.11.0 was never re-published.

Adds workflow_dispatch and adds the workflow file itself to the push path filter so future workflow edits self-trigger.

Summary by CodeRabbit

  • Chores
    • Updated Docker workflow configuration to support manual triggering and rebuild when workflow files change.

Review Change Stack

Hotfix #866 changed only the workflow file, so docker-publish did not
retrigger and v2.11.0 was never re-published. Add workflow_dispatch and
include the workflow file in the push path filter.
@vercel

vercel Bot commented May 15, 2026 •

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
lucky Ready Ready Preview, Comment May 15, 2026 5:57pm

Request Review

@greptile-apps greptile-apps Bot 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.

LucasSantana-Dev has reached the 50-review limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@github-actions

Copy link
Copy Markdown

Failed to generate code suggestions for PR

@coderabbitai

coderabbitai Bot commented May 15, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 5fab85ec-e9ef-4d59-8bfc-37fa66a84376

📥 Commits

Reviewing files that changed from the base of the PR and between e6267f5 and 2100042.

📒 Files selected for processing (1)
  • .github/workflows/docker-publish.yml

📝 Walkthrough

Walkthrough

The Docker publish workflow now supports both automatic and manual triggering. A workflow_dispatch trigger enables developers to run the workflow manually from the GitHub Actions UI, and the push.paths filter is extended to include the workflow file itself, so changes to the workflow configuration automatically trigger a build-and-push job.

Changes

Docker Publish Workflow Triggers

Layer / File(s) Summary
Workflow dispatch and self-triggered builds
.github/workflows/docker-publish.yml
Added workflow_dispatch trigger for manual invocation and extended on.push.paths filter to include .github/workflows/docker-publish.yml so the workflow rebuilds when its own configuration changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • LucasSantana-Dev/Lucky#437: Both PRs modify GitHub Actions workflow behavior around the docker-publish.yml job—main adds manual triggering and adjusts push paths to control when docker-publish runs, while the retrieved PR changes deploy.yml to wait for the corresponding docker-publish.yml run for the same commit before firing the webhook.
  • LucasSantana-Dev/Lucky#462: The main PR updates docker-publish.yml trigger conditions (including self-path and manual dispatch), which directly impacts whether the deploy workflow needs to wait for a docker-publish run—the same scenario addressed by the retrieved PR's deploy.yml logic to skip waiting when Docker-relevant changes aren't present.

Suggested labels

ci, size/s

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main changes: adding workflow_dispatch trigger and configuring retrigger on workflow file edits.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/docker-publish-triggers

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sonarqubecloud

Copy link
Copy Markdown

@LucasSantana-Dev
LucasSantana-Dev merged commit 7a959f8 into main May 15, 2026
28 checks passed

This branch was successfully deployed

1 active deployment
Preview — 2100042d Deployed May 15, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant