Skip to content

Commit 1e504b4

Browse files
committed
ci: run integration tests on pull requests into main
1 parent ce3d5c5 commit 1e504b4

2 files changed

Lines changed: 9 additions & 3 deletions

File tree

‎.github/workflows/integration.yml‎

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,16 @@
11
# Integration tests against a live test wiki. Credentials come from the
22
# INTEGR_TEST_* secrets; where they are unavailable (e.g. forks), the
33
# credential fixtures make the tests skip instead of fail.
4+
#
5+
# Runs on every pull request into main, so breaking changes surface before the
6+
# release job (gated separately by required review on the pypi environment)
7+
# ever starts. No push trigger: a push to main is already covered by the PR
8+
# that preceded it, so a second run would add nothing.
49

510
name: Integration
611

712
on:
8-
push:
13+
pull_request:
914
branches: [main]
1015
workflow_dispatch:
1116

‎docs/dev.md‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,8 +62,9 @@ excluded by default:
6262
### Integration tests
6363

6464
Integration tests run against a live wiki and need credentials; tests
65-
whose credentials are missing skip instead of fail. CI runs them on
66-
every push to `main` (workflow `integration.yml`).
65+
whose credentials are missing skip instead of fail. CI runs them on every
66+
pull request into `main` (workflow `integration.yml`), so they gate the
67+
release rather than run after it.
6768

6869
=== "make"
6970

0 commit comments

Comments
 (0)