Skip to content

Commit

Permalink
Merge pull request #22 from 10up/fix/16
Browse files Browse the repository at this point in the history
Add workflow to run PHP compat testing
  • Loading branch information
dkotter authored Oct 18, 2023
2 parents 7edd4a6 + c8680cb commit eed703e
Show file tree
Hide file tree
Showing 4 changed files with 73 additions and 1 deletion.
45 changes: 45 additions & 0 deletions .github/workflows/php-compat.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,45 @@
name: PHP Compatibility

on:
push:
branches:
- develop
- trunk
paths:
- '**.php'
pull_request:
branches:
- develop
paths:
- '**.php'

jobs:
php-compatibility:
runs-on: ubuntu-latest

steps:
- uses: actions/checkout@v4
with:
fetch-depth: 0

- name: Setup proper PHP version
uses: shivammathur/setup-php@v2
with:
php-version: 7.4

- name: Get composer cache directory
id: composer-cache
run: echo "dir=$(composer config cache-files-dir)" >> $GITHUB_OUTPUT

- name: Cache dependencies
uses: actions/cache@v3
with:
path: ${{ steps.composer-cache.outputs.dir }}
key: ${{ runner.os }}-composer-${{ hashFiles('**/composer.lock') }}
restore-keys: ${{ runner.os }}-composer-

- name: Install dependencies
run: composer install --prefer-dist

- name: Run PHP Compatibility
run: ./vendor/bin/phpcs --standard=./phpcs-compat.xml -p .
2 changes: 1 addition & 1 deletion composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
}
],
"require": {
"php": ">=7.2"
"php": ">=7.4"
},
"require-dev": {
"10up/phpcs-composer": "dev-master",
Expand Down
19 changes: 19 additions & 0 deletions phpcs-compat.xml
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
<?xml version="1.0"?>
<ruleset name="PHPCompatibilityWP">
<description>PHP Compatibility checks only</description>

<!-- Files to check -->
<arg name="extensions" value="php"/>
<file>.</file>

<rule ref="PHPCompatibilityWP" />

<exclude-pattern>*/dist/*</exclude-pattern>
<exclude-pattern>*/languages/*</exclude-pattern>
<exclude-pattern>*/tests/*</exclude-pattern>
<exclude-pattern>*/node_modules/*</exclude-pattern>
<exclude-pattern>*/vendor/*</exclude-pattern>

<config name="minimum_supported_wp_version" value="5.7"/>
<config name="testVersion" value="7.4-"/>
</ruleset>
8 changes: 8 additions & 0 deletions phpcs.xml
Original file line number Diff line number Diff line change
Expand Up @@ -5,4 +5,12 @@
<rule ref="10up-Default" />
<exclude-pattern>*/dist/*</exclude-pattern>
<exclude-pattern>*/tests/*</exclude-pattern>

<config name="minimum_supported_wp_version" value="5.7"/>
<config name="testVersion" value="7.4-"/>

<!-- Exclude the PHPCompatibilityWP ruleset -->
<rule ref="PHPCompatibilityWP">
<exclude name="PHPCompatibilityWP"/>
</rule>
</ruleset>

0 comments on commit eed703e

Please sign in to comment.