Skip to content

Commit

Permalink
Merge pull request #26 from kiddivouchers/update-ci
Browse files Browse the repository at this point in the history
Update CI configuration
  • Loading branch information
la5942 authored Nov 1, 2023
2 parents bc9d106 + aa91bb2 commit d1e6d59
Show file tree
Hide file tree
Showing 2 changed files with 24 additions and 7 deletions.
25 changes: 21 additions & 4 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ on:
jobs:
build:
name: PHP ${{ matrix.php-version }} (ZTS ${{ matrix.php-zts }})
runs-on: ubuntu-20.04
runs-on: ubuntu-22.04
timeout-minutes: 30
strategy:
fail-fast: false
Expand All @@ -24,9 +24,15 @@ jobs:
- '8.0'
- '8.1'
- '8.2'
- '8.3'
php-zts:
- nts
# @todo Add ts when setup-php supports ZTS on Ubuntu
- ts
exclude:
- php-version: '5.6'
php-zts: 'ts'
- php-version: '8.3'
php-zts: 'ts'
steps:
- name: Setup PHP
uses: shivammathur/setup-php@v2
Expand All @@ -35,6 +41,9 @@ jobs:
coverage: none
env:
phpts: ${{ matrix.php-zts }}
- name: PHP version
run: |
php --version
- name: Set PHP_API
run: |
echo "PHP_API=$(php-config --phpapi)" >> $GITHUB_ENV
Expand All @@ -46,15 +55,23 @@ jobs:
./configure
make
make test REPORT_EXIT_STATUS=1 NO_INTERACTION=1 TESTS="--show-all"
- name: Save artifact
- name: Save artifact (NTS)
uses: actions/upload-artifact@v3
if: matrix.php-zts == 'nts'
with:
name: timecop_${{ env.PHP_API }}.so
path: modules/timecop.so
if-no-files-found: error
- name: Save artifact (TS)
uses: actions/upload-artifact@v3
if: matrix.php-zts == 'ts'
with:
name: timecop_${{ env.PHP_API }}_zts.so
path: modules/timecop.so
if-no-files-found: error
package:
name: Package for download
runs-on: ubuntu-20.04
runs-on: ubuntu-22.04
timeout-minutes: 10
needs: build
steps:
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/valgrind.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,10 +15,10 @@ jobs:
php-version:
- '5.6.40'
- '7.4.19'
- '8.1.13'
- '8.2.0'
- '8.1.25'
- '8.2.12'
os:
- 'ubuntu-20.04'
- 'ubuntu-22.04'
steps:
- name: Install valgrind
run: |
Expand Down

0 comments on commit d1e6d59

Please sign in to comment.