Skip to content

refactor: split dev build into separate workflow that only runs on main#239

Merged
d0ugal merged 1 commit intomainfrom
refactor/split-dev-build-workflow
Nov 28, 2025
Merged

refactor: split dev build into separate workflow that only runs on main#239
d0ugal merged 1 commit intomainfrom
refactor/split-dev-build-workflow

Conversation

@d0ugal
Copy link
Copy Markdown
Owner

@d0ugal d0ugal commented Nov 28, 2025

Problem

Dev builds were running for release PRs and release-please branches, which is unnecessary since release builds are handled by the release workflow.

Solution

Split the dev-build job into its own workflow file that only triggers on push to main branch. This ensures dev builds never run for:

  • PRs (including release PRs)
  • Release-please branches
  • Any branch other than main

Changes

  • Created new .github/workflows/dev-build.yml workflow
    • Only triggers on push to main branch
    • Skips release commits automatically
    • Skips documentation-only changes
  • Removed dev-build and check-build-needed jobs from ci.yml
  • Simplified CI workflow to focus on test, lint, build, and security

Benefits

  • Clearer separation of concerns: Dev builds are isolated in their own workflow
  • Impossible to run on PRs: The workflow only triggers on push to main
  • Simpler CI: Main CI workflow is cleaner and easier to understand
  • No more release PR builds: Release PRs can't trigger dev builds since they're not pushes to main

Move dev-build job to its own workflow file that only triggers on push to main.
This ensures dev builds never run for:
- PRs (including release PRs)
- Release-please branches
- Any branch other than main

Benefits:
- Clearer separation of concerns
- Simpler CI workflow
- Impossible for dev builds to run on PRs or release branches
@d0ugal d0ugal enabled auto-merge (rebase) November 28, 2025 21:26
@d0ugal d0ugal merged commit 9cfd68d into main Nov 28, 2025
6 checks passed
@d0ugal d0ugal deleted the refactor/split-dev-build-workflow branch November 28, 2025 21:26
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.

1 participant