Skip to content

Commit a3a4f27

Browse files
rlerdorfclaude
andcommitted
CI: use shivammathur/setup-php for all PHP versions
Test 7.2 through 8.6 with a single shivammathur/setup-php matrix instead of the docker-image based setup, which had no published php:8.6 image. 8.6 is built from php-src master; fail-fast is disabled so a failure there does not cancel the other versions. The package.xml validation previously done in ci/test_inner.sh is kept as a "pecl package" step. The now-unused ci/ docker scripts are removed. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent 583843e commit a3a4f27

4 files changed

Lines changed: 21 additions & 53 deletions

File tree

.github/workflows/main.yml

Lines changed: 21 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -22,26 +22,32 @@ jobs:
2222

2323
# See https://docs.github.com/en/actions/reference/workflow-syntax-for-github-actions#using-environment-variables-in-a-matrix
2424
strategy:
25+
# 8.6 is built from php-src master, so a failure there should not cancel
26+
# the other versions.
2527
fail-fast: false
2628
matrix:
27-
include:
28-
# NOTE: If this is not quoted, the yaml parser will convert 8.0 to the number 8,
29-
# and the docker image `php:8` is the latest minor version of php 8.x (8.1).
30-
- PHP_VERSION: '7.2'
31-
- PHP_VERSION: '7.3'
32-
- PHP_VERSION: '7.4'
33-
- PHP_VERSION: '8.0'
34-
- PHP_VERSION: '8.1'
35-
- PHP_VERSION: '8.2'
36-
- PHP_VERSION: '8.3'
37-
- PHP_VERSION: '8.4'
38-
- PHP_VERSION: '8.5'
29+
# NOTE: Quoted so the yaml parser does not turn e.g. 8.0 into the number 8.
30+
PHP_VERSION: ['7.2', '7.3', '7.4', '8.0', '8.1', '8.2', '8.3', '8.4', '8.5', '8.6']
3931

4032
# Steps represent a sequence of tasks that will be executed as part of the job
4133
steps:
4234
# Checks-out your repository under $GITHUB_WORKSPACE, so your job can access it
4335
- uses: actions/checkout@v2
4436

45-
# Runs a single command using the runners shell
46-
- name: Build and test in docker
47-
run: bash ci/test_dockerized.sh ${{ matrix.PHP_VERSION }}
37+
- name: Setup PHP
38+
uses: shivammathur/setup-php@v2
39+
with:
40+
php-version: ${{ matrix.PHP_VERSION }}
41+
tools: pecl
42+
43+
- name: Build
44+
run: |
45+
phpize
46+
EXTRA_CFLAGS='-Wall -Wextra -Wno-unused-parameter' ./configure
47+
make -j2
48+
49+
- name: Test
50+
run: NO_INTERACTION=1 REPORT_EXIT_STATUS=1 make test TESTS="--show-diff"
51+
52+
- name: Verify package.xml is valid
53+
run: pecl package

ci/Dockerfile

Lines changed: 0 additions & 13 deletions
This file was deleted.

ci/test_dockerized.sh

Lines changed: 0 additions & 16 deletions
This file was deleted.

ci/test_inner.sh

Lines changed: 0 additions & 9 deletions
This file was deleted.

0 commit comments

Comments
 (0)