Skip to content

Commit

Permalink
Split tests and static code analysis in CI
Browse files Browse the repository at this point in the history
  • Loading branch information
sprain committed Nov 3, 2023
1 parent c992a86 commit 8944021
Showing 1 changed file with 35 additions and 7 deletions.
42 changes: 35 additions & 7 deletions .github/workflows/tests.yml → .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ on:
pull_request:

jobs:
linux_tests:
tests:
runs-on: ubuntu-20.04

strategy:
Expand All @@ -27,17 +27,45 @@ jobs:
tools: composer:v2
coverage: none

- name: Check for syntax errors in PHP files
run: find ./ -type f -name '*.php' -print0 | xargs -0 -L1 -P4 -- php -l

- name: Check composer files
run: composer validate --strict

- name: Install dependencies
run: composer update --${{ matrix.stability }} --prefer-dist --no-interaction --no-progress

- name: Execute tests
run: vendor/bin/phpunit --verbose


code_checks:
runs-on: ubuntu-20.04
name: Static code analysis

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

- name: Setup PHP
uses: shivammathur/setup-php@v2
with:
php-version: '8.2'
extensions: dom, curl, libxml, mbstring, zip, bcmath, gd
tools: composer
coverage: none

- 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 --no-progress

- name: Check composer files
run: composer validate --strict

- name: Check coding standard
run: vendor/bin/php-cs-fixer --no-interaction --dry-run --diff -v fix lib/

0 comments on commit 8944021

Please sign in to comment.