Skip to content

Eliminate duplicate workflow runs on pull requests#22

Merged
RadCod3 merged 2 commits into
mainfrom
fix-workflows
Jan 6, 2026
Merged

Eliminate duplicate workflow runs on pull requests#22
RadCod3 merged 2 commits into
mainfrom
fix-workflows

Conversation

@RadCod3
Copy link
Copy Markdown
Owner

@RadCod3 RadCod3 commented Jan 6, 2026

Problem

Opening a PR triggered workflows multiple times:

  • CI workflow ran twice (once on push to feature branch, once on pull_request event)
  • Docker publish workflow ran unnecessarily on PRs (already covered by CI's Docker build test)

Solution

  • CI workflow: Removed push trigger for feature branches (feat/, fix/, chore/**). Now only runs on PRs to main and pushes to main.
  • Docker publish workflow: Removed pull_request trigger. Now only runs on pushes to main and version tags.

Copilot AI review requested due to automatic review settings January 6, 2026 17:10
@RadCod3 RadCod3 merged commit 3916193 into main Jan 6, 2026
7 checks passed
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR eliminates duplicate workflow runs when opening pull requests by adjusting GitHub Actions triggers. The CI workflow previously ran twice on feature branch PRs (once on push, once on pull_request), and the Docker publish workflow ran unnecessarily on PRs despite CI already testing Docker builds.

Key Changes:

  • CI workflow now only runs on pull requests to main and direct pushes to main (removed feature branch push triggers)
  • Docker publish workflow now only runs on pushes to main and version tags (removed pull_request trigger)
  • Docker Compose configuration updated to use published image instead of local build

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/ci.yml Removed push triggers for feature branches (feat/, fix/, chore/**), preventing duplicate runs on PRs
.github/workflows/docker-publish.yml Removed pull_request trigger, as Docker builds are already tested in CI workflow
docker-compose.yml Changed from local build to using published ghcr.io image

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docker-compose.yml
@RadCod3 RadCod3 deleted the fix-workflows branch January 8, 2026 16:46
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