Skip to content

Commit

Permalink
Split package validation out to its own check, that should only run o…
Browse files Browse the repository at this point in the history
…n master.
  • Loading branch information
Danack committed Nov 2, 2022
1 parent c555f5e commit d719de5
Show file tree
Hide file tree
Showing 3 changed files with 37 additions and 1 deletion.
1 change: 0 additions & 1 deletion .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,6 @@ jobs:

- name: Sanity check package.xml and install ImageMagick
run: |
bash validate_package.sh
for file in tests/*.phpt; do grep $(basename $file) package.xml >/dev/null || (echo "Missing $file from package.xml" ; exit 1); done
bash ./imagemagick_dependency.sh "${{ matrix.imagemagick }}" $(pwd)
Expand Down
36 changes: 36 additions & 0 deletions .github/workflows/release_validate.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,36 @@
name: Validate pecl package

on:
push:
pull_request:
types: [opened, synchronize, reopened]
release:
types: [created]

jobs:
test_package_valid:
name: basic PHP ${{ matrix.php }}
runs-on: ubuntu-20.04
if: github.ref == 'refs/heads/master'
strategy:
fail-fast: false
matrix:
os: [ ubuntu-latest ]
php: [ 5.4 ]

steps:
- name: Checkout code
uses: actions/checkout@v2

- name: Setup PHP
uses: shivammathur/setup-php@v2
with:
php-version: ${{ matrix.php }}

- name: Sanity check package.xml and install ImageMagick
run: |
bash validate_package.sh
for file in tests/*.phpt; do grep $(basename $file) package.xml >/dev/null || (echo "Missing $file from package.xml" ; exit 1); done
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -70,3 +70,4 @@ tests/*.php
tests/*.sh
bugReports
data
*.dep

0 comments on commit d719de5

Please sign in to comment.