Skip to content

ci: prefer PR-driven triggers and deduplicate runs via concurrency group - #76

Closed
MLopez-Ibanez with Copilot wants to merge 2 commits into
mainfrom
copilot/update-github-actions-workflows
Closed

ci: prefer PR-driven triggers and deduplicate runs via concurrency group#76
MLopez-Ibanez with Copilot wants to merge 2 commits into
mainfrom
copilot/update-github-actions-workflows

Conversation

Copilot AI commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Force-pushes to PR branches were not reliably triggering CI because push triggers exclude feature branches, and pull_request triggers had no mechanism to cancel stale runs. The result was silent CI gaps on active PRs.

Changes

  • pull_request before push in C.yml, python.yml, R.yml, matlab.yml — PRs are the primary CI signal for feature branches; ordering reflects intent.
  • push retained for main and v[0-9]+* only — CI still runs on merge and release branches; python.yml retains its release: trigger for wheel publishing.
  • Concurrency group updated across all four workflows:
# Before
group: ${{ github.workflow }}-${{ github.ref }}

# After — deduplicates by PR number; force-pushes cancel stale runs
group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}

docs-preview-cleanup.yml, rchk.yml, and revdepcheck.yml are unchanged — they use workflow_call/workflow_dispatch/closed-PR triggers that don't follow this pattern.

Co-authored-by: MLopez-Ibanez <2620021+MLopez-Ibanez@users.noreply.github.com>
Copilot AI changed the title [WIP] Update GitHub Actions workflows to prioritize PR triggers ci: prefer PR-driven triggers and deduplicate runs via concurrency group Aug 6, 2026
Copilot AI requested a review from MLopez-Ibanez August 6, 2026 23:07
@MLopez-Ibanez
MLopez-Ibanez deleted the copilot/update-github-actions-workflows branch August 6, 2026 23:17
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