From a18506ef25e16602bf8922bf5d79146525b28e2b Mon Sep 17 00:00:00 2001 From: jrfnl Date: Tue, 21 Sep 2021 02:53:26 +0200 Subject: [PATCH] Testing setup PHP: using parallel-lint tool --- .github/workflows/tests.yml | 20 +++++++++----------- 1 file changed, 9 insertions(+), 11 deletions(-) diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml index 1880899511..65a18c4bfc 100644 --- a/.github/workflows/tests.yml +++ b/.github/workflows/tests.yml @@ -48,29 +48,27 @@ jobs: name: "Lint: PHP ${{ matrix.php }}" continue-on-error: ${{ matrix.experimental }} + env: + PL_CLI_ARGS: . -e php,phps --exclude vendor --exclude .git --exclude build + steps: - name: Checkout code uses: actions/checkout@v2 - name: Install PHP - uses: shivammathur/setup-php@v2 + uses: shivammathur/setup-php@develop with: php-version: ${{ matrix.php }} coverage: none - tools: cs2pr - - # Install dependencies and handle caching in one go. - # @link https://github.com/marketplace/actions/install-composer-dependencies - - name: Install Composer dependencies - uses: "ramsey/composer-install@v1" + tools: cs2pr, parallel-lint - name: Lint against parse errors - if: ${{ matrix.php != '8.1' }} - run: composer lint -- --checkstyle | cs2pr + if: ${{ matrix.php != '8.1' && matrix.php != '7.2' }} + run: parallel-lint ${{ env.PL_CLI_ARGS }} --checkstyle | cs2pr - name: Lint against parse errors (PHP 8.1) - if: ${{ matrix.php == '8.1' }} - run: composer lint + if: ${{ matrix.php == '8.1' || matrix.php == '7.2' }} + run: parallel-lint ${{ env.PL_CLI_ARGS }} test: needs: ['coding-standard', 'lint']