Skip to content

ci: trigger all build jobs on every push - #9

Merged
evildevill merged 2 commits into
mainfrom
copilot/automate-build-linux-appimage
Jul 25, 2026
Merged

ci: trigger all build jobs on every push#9
evildevill merged 2 commits into
mainfrom
copilot/automate-build-linux-appimage

Conversation

Copilot AI commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

Build jobs (AppImage, .deb, Windows installer, macOS DMG) previously only ran on version tags or manual workflow_dispatch, requiring manual intervention for every non-tag push.

Changes

  • All four build jobs (build-linux-appimage, build-linux-deb, build-windows, build-macos): updated if condition to include github.event_name == 'push', triggering on every push to any branch or tag
if: github.event_name == 'push' || startsWith(github.ref, 'refs/tags/v') || (github.event_name == 'workflow_dispatch' && inputs.build_artifacts)

The release job remains tag-only (startsWith(github.ref, 'refs/tags/v')).

@evildevill
evildevill marked this pull request as ready for review July 25, 2026 09:36
Copilot AI review requested due to automatic review settings July 25, 2026 09:36
@evildevill
evildevill merged commit 757cb0e into main Jul 25, 2026

Copilot AI 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.

Pull request overview

Updates the CI workflow so platform build artifact jobs run automatically on push events, reducing the need for manual workflow_dispatch runs during normal development.

Changes:

  • Updated all four build jobs (Linux AppImage, Linux .deb, Windows installer, macOS DMG) to include github.event_name == 'push' in their if conditions.
  • Kept the release job gated to version tags (refs/tags/v*).
Comments suppressed due to low confidence (3)

.github/workflows/ci.yml:114

  • Same as above: startsWith(github.ref, 'refs/tags/v') is redundant once github.event_name == 'push' is included (tag pushes are already limited to v* by the workflow trigger). Simplifying reduces repetition and makes the intent clearer.
    if: github.event_name == 'push' || startsWith(github.ref, 'refs/tags/v') || (github.event_name == 'workflow_dispatch' && inputs.build_artifacts)

.github/workflows/ci.yml:137

  • Same as above: since github.event_name == 'push' already covers tag pushes allowed by on.push.tags, the explicit startsWith(github.ref, 'refs/tags/v') check is redundant and can be removed to keep the condition readable.
    if: github.event_name == 'push' || startsWith(github.ref, 'refs/tags/v') || (github.event_name == 'workflow_dispatch' && inputs.build_artifacts)

.github/workflows/ci.yml:167

  • Same as above: with github.event_name == 'push' included, the startsWith(github.ref, 'refs/tags/v') clause is redundant given the workflow trigger already restricts tag pushes to v*. Removing it avoids duplicated logic.
    if: github.event_name == 'push' || startsWith(github.ref, 'refs/tags/v') || (github.event_name == 'workflow_dispatch' && inputs.build_artifacts)

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

Comment thread .github/workflows/ci.yml
name: Build Linux (AppImage)
needs: [lint, test]
if: startsWith(github.ref, 'refs/tags/v') || (github.event_name == 'workflow_dispatch' && inputs.build_artifacts)
if: github.event_name == 'push' || startsWith(github.ref, 'refs/tags/v') || (github.event_name == 'workflow_dispatch' && inputs.build_artifacts)
Comment thread .github/workflows/ci.yml
name: Build Linux (AppImage)
needs: [lint, test]
if: startsWith(github.ref, 'refs/tags/v') || (github.event_name == 'workflow_dispatch' && inputs.build_artifacts)
if: github.event_name == 'push' || startsWith(github.ref, 'refs/tags/v') || (github.event_name == 'workflow_dispatch' && inputs.build_artifacts)
Comment thread .github/workflows/ci.yml
name: Build Linux (AppImage)
needs: [lint, test]
if: startsWith(github.ref, 'refs/tags/v') || (github.event_name == 'workflow_dispatch' && inputs.build_artifacts)
if: github.event_name == 'push' || startsWith(github.ref, 'refs/tags/v') || (github.event_name == 'workflow_dispatch' && inputs.build_artifacts)
Copilot stopped work on behalf of evildevill due to an error July 25, 2026 09:42
Copilot stopped work on behalf of evildevill due to an error July 25, 2026 09:42
@evildevill
evildevill deleted the copilot/automate-build-linux-appimage branch July 25, 2026 13:18
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.

3 participants