Skip to content

Commit d789cf6

Browse files
authored
Merge pull request #25 from script-development/wr-0796-release-lane-coverage-parity
ci(release): run the real coverage gate on tagged commits (WR-0796)
2 parents b93d532 + fefff03 commit d789cf6

1 file changed

Lines changed: 12 additions & 6 deletions

File tree

.github/workflows/release.yml

Lines changed: 12 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,16 @@
11
# Release workflow — runs on tag push (v*).
22
#
3-
# Composer packages on public packagist.org auto-sync from the configured Git
4-
# repository via push-event webhook (https://packagist.org/api/github), so this
5-
# workflow's primary job is to (1) re-run the CI gates on the tagged commit and
6-
# (2) create a GitHub release pointing at the changelog entry.
3+
# `verify` re-runs the repository's gates against the tagged commit — audit,
4+
# format, static analysis, and the coverage suite — and `release` then creates
5+
# a GitHub release pointing at the changelog entry.
6+
#
7+
# WR-0796: `coverage: pcov` and `composer test:coverage` are a matched pair.
8+
# Dropping the script silently removes the --min=95 coverage gate from tagged
9+
# builds; dropping the driver fails the job outright ("No code coverage driver
10+
# is available"). Change both or neither.
711
#
12+
# Composer packages on public packagist.org auto-sync from the configured Git
13+
# repository via push-event webhook (https://packagist.org/api/github).
814
# First-time submission to packagist.org is a manual user step (needs the
915
# Packagist login); once the package is registered and the GitHub service hook
1016
# is enabled, this tag push auto-syncs the versioned release.
@@ -30,7 +36,7 @@ jobs:
3036
uses: shivammathur/setup-php@728c6c6b8cf02c2e48117716a91ee48313958a19 # v2
3137
with:
3238
php-version: ${{ matrix.php }}
33-
coverage: none
39+
coverage: pcov
3440
tools: composer:v2
3541

3642
- name: Install dependencies
@@ -46,7 +52,7 @@ jobs:
4652
run: composer phpstan
4753

4854
- name: Tests
49-
run: composer test
55+
run: composer test:coverage
5056

5157
release:
5258
needs: verify

0 commit comments

Comments
 (0)