jessey/rigdoctor
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 59 Wiki Activity

ci: run tests.yml on pull_request only (no push) to avoid double runs

Browse Source 
A branch with an open PR triggered both the push and pull_request events, running
every job twice. Trigger on pull_request only; pushes to main are already tested
by release.yml's `test` job. No version bump (CI config only).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
Jessey van Offeren
2026-05-22 14:42:41 +02:00
parent 2ee7763d00
commit 2989e8e23e
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.gitea/workflows/tests.yml
+4 -3
View File
@@ -1,14 +1,15 @@
name: tests name: tests
run-name: Run test suite run-name: Run test suite
# Runs the unittest suite on every push and pull request. Two jobs: # Runs the unittest suite on pull requests (once per PR). Pushes to main are covered by the
# `test` job in release.yml, so we don't trigger on push here — that would double every run.
# Two jobs:
# core — stdlib-only install; the GUI tests skip (@skipUnless HAVE_QT). Bulletproof. # core — stdlib-only install; the GUI tests skip (@skipUnless HAVE_QT). Bulletproof.
# gui-smoke — installs the GUI extra + offscreen Qt libs and runs the same suite headless, # gui-smoke — installs the GUI extra + offscreen Qt libs and runs the same suite headless,
# exercising the MainWindow/SetupWizard/DiagnosticDialog construction tests. # exercising the MainWindow/SetupWizard/DiagnosticDialog construction tests.
# Make `core` a required status check on `main` so a PR can't merge with failing tests. # Make `tests / core (pull_request)` a required status check on `main` so a PR can't merge red.
on: on:
push:
pull_request: pull_request:
jobs: jobs: