CS: improve use statements #307
Workflow file for this run
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| name: Test | |
| on: | |
| # Run on pushes to `master` and on all pull requests. | |
| # Prevent the "push" build from running when there are only irrelevant changes. | |
| push: | |
| branches: | |
| - master | |
| paths-ignore: | |
| - '**.md' | |
| pull_request: | |
| # Allow manually triggering the workflow. | |
| workflow_dispatch: | |
| # Cancels all previous workflow runs for the same branch that have not yet completed. | |
| concurrency: | |
| # The concurrency group contains the workflow name and the branch name. | |
| group: ${{ github.workflow }}-${{ github.ref }} | |
| cancel-in-progress: true | |
| jobs: | |
| #### PHP LINT STAGE #### | |
| # Linting against high/low PHP versions should catch everything. | |
| # If needs be, we can always add interim versions at a later point in time. | |
| lint: | |
| runs-on: ubuntu-latest | |
| strategy: | |
| matrix: | |
| php: ['5.4', 'latest', '8.3'] | |
| name: "Lint: PHP ${{ matrix.php }}" | |
| continue-on-error: ${{ matrix.php == '8.3' }} | |
| steps: | |
| - name: Checkout code | |
| uses: actions/checkout@v3 | |
| - name: Install PHP | |
| uses: shivammathur/setup-php@v2 | |
| with: | |
| php-version: ${{ matrix.php }} | |
| coverage: none | |
| tools: cs2pr | |
| - name: Install Composer dependencies | |
| uses: "ramsey/composer-install@v2" | |
| with: | |
| # Bust the cache at least once a month - output format: YYYY-MM. | |
| custom-cache-suffix: $(date -u "+%Y-%m") | |
| - name: Lint against parse errors | |
| run: ./bin/php-lint --checkstyle | cs2pr | |
| test: | |
| # No use running the tests if there is a linting error somewhere as they would fail anyway. | |
| needs: lint | |
| runs-on: ubuntu-latest | |
| strategy: | |
| # Keys: | |
| # - php: The PHP versions to test against. | |
| # - phpcs_version: The PHPCS versions to test against. | |
| # IMPORTANT: test runs shouldn't fail because of PHPCS being incompatible with a PHP version. | |
| # - PHPCS will run without errors on PHP 5.4 - 7.4 on any supported version. | |
| # - PHP 8.0 needs PHPCS 3.5.7+ to run without errors, and we require a higher minimum version. | |
| # - PHP 8.1 needs PHPCS 3.6.1+ to run without errors, but works best with 3.7.1+, and we require at least this minimum version. | |
| # - The `wpcs_version` key is added to allow additional test builds when multiple WPCS versions | |
| # would be supported. As, at this time, only the latest stable release of WPCS is supported, | |
| # no additional versions are included in the array. | |
| matrix: | |
| php: ['5.4', '5.5', '5.6', '7.0', '7.1', '7.2', '7.3', '7.4', '8.0', '8.1', '8.2'] | |
| phpcs_version: ['3.7.1', 'dev-master'] | |
| wpcs_version: ['2.3.*'] | |
| include: | |
| - php: '8.3' | |
| phpcs_version: 'dev-master' | |
| wpcs_version: '2.3.*' | |
| name: "Test: PHP ${{ matrix.php }} - PHPCS ${{ matrix.phpcs_version }} - WPCS ${{ matrix.wpcs_version }}" | |
| continue-on-error: ${{ matrix.php == '8.3' }} | |
| steps: | |
| - name: Checkout code | |
| uses: actions/checkout@v3 | |
| # On stable PHPCS versions, allow for PHP deprecation notices. | |
| # Unit tests don't need to fail on those for stable releases where those issues won't get fixed anymore. | |
| # Note: the "elif" condition is temporary and should be removed once VIPCS updates to WPCS 3.0+. | |
| - name: Setup ini config | |
| id: set_ini | |
| run: | | |
| if [[ "${{ matrix.phpcs_version }}" != "dev-master" ]]; then | |
| echo 'PHP_INI=error_reporting=E_ALL & ~E_DEPRECATED' >> $GITHUB_OUTPUT | |
| elif [[ "${{ matrix.php }}" == "8.1" ]]; then | |
| echo 'PHP_INI=error_reporting=E_ALL & ~E_DEPRECATED' >> $GITHUB_OUTPUT | |
| else | |
| echo 'PHP_INI=error_reporting=-1' >> $GITHUB_OUTPUT | |
| fi | |
| - name: Install PHP | |
| uses: shivammathur/setup-php@v2 | |
| with: | |
| php-version: ${{ matrix.php }} | |
| ini-values: ${{ steps.set_ini.outputs.PHP_INI }} | |
| coverage: none | |
| - name: 'Composer: set PHPCS version for tests' | |
| run: composer require squizlabs/php_codesniffer:"${{ matrix.phpcs_version }}" --no-update --no-scripts --no-interaction | |
| - name: 'Composer: set WPCS version for tests' | |
| run: composer require wp-coding-standards/wpcs:"${{ matrix.wpcs_version }}" --no-update --no-scripts --no-interaction | |
| # Install dependencies and handle caching in one go. | |
| # @link https://github.com/marketplace/actions/install-composer-dependencies | |
| - name: Install Composer dependencies - normal | |
| if: ${{ startsWith( matrix.php, '8' ) == false }} | |
| uses: "ramsey/composer-install@v2" | |
| with: | |
| # Bust the cache at least once a month - output format: YYYY-MM. | |
| custom-cache-suffix: $(date -u "+%Y-%m") | |
| # PHPUnit 7.x does not allow for installation on PHP 8, so ignore platform | |
| # requirements to get PHPUnit 7.x to install on nightly. | |
| - name: Install Composer dependencies - with ignore platform | |
| if: ${{ startsWith( matrix.php, '8' ) }} | |
| uses: "ramsey/composer-install@v2" | |
| with: | |
| composer-options: --ignore-platform-reqs | |
| custom-cache-suffix: $(date -u "+%Y-%m") | |
| - name: Run the unit tests | |
| run: ./bin/unit-tests | |
| - name: Run the ruleset tests | |
| run: ./bin/ruleset-tests |