From 6c8fbdd3855d029a2cd3b6f7fb2a165ae7bdef6e Mon Sep 17 00:00:00 2001 From: marcusquinn <6428977+marcusquinn@users.noreply.github.com> Date: Mon, 16 Mar 2026 21:50:10 +0000 Subject: [PATCH] fix: remove redundant composer require steps in code-quality workflow szepeviktor/phpstan-wordpress, wp-coding-standards/wpcs, and dealerdirect/phpcodesniffer-composer-installer are already declared in composer.json require-dev. The extra 'composer require' steps after 'composer install' caused a second packagist.org network hit that timed out in CI, failing the PHPStan Static Analysis job. Fixes the Code Quality CI failure on PR #51 (issue #45). --- .github/workflows/code-quality.yml | 8 -------- 1 file changed, 8 deletions(-) diff --git a/.github/workflows/code-quality.yml b/.github/workflows/code-quality.yml index e97c3ff..ca07297 100644 --- a/.github/workflows/code-quality.yml +++ b/.github/workflows/code-quality.yml @@ -29,11 +29,6 @@ jobs: - name: Install dependencies run: composer install --prefer-dist --no-progress - - name: Install WordPress Coding Standards - run: | - composer require --dev wp-coding-standards/wpcs dealerdirect/phpcodesniffer-composer-installer - vendor/bin/phpcs --config-set installed_paths vendor/wp-coding-standards/wpcs - - name: Run PHPCS run: composer phpcs continue-on-error: true @@ -61,9 +56,6 @@ jobs: - name: Install dependencies run: composer install --prefer-dist --no-progress - - name: Install PHPStan WordPress stubs - run: composer require --dev szepeviktor/phpstan-wordpress - - name: Run PHPStan run: composer phpstan continue-on-error: true