Skip to content

Commit 59a9ec8

Browse files
committed
GH Actions: add builds against PHP 8.3 and 8.4
As things were, the CI was still allowing PHP 8.2 to fail. This is no longer necessary since PR 34 has been merged. Additionally, now PR 77 has been merged, the build should also pass against PHP 8.3. Adding 8.4 as well as an early warning system, but allowing it to fail. Includes updating the badge in the README.
1 parent d11632a commit 59a9ec8

File tree

2 files changed

+13
-4
lines changed

2 files changed

+13
-4
lines changed

.github/workflows/test.yml

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ jobs:
2222

2323
strategy:
2424
matrix:
25-
php: ['5.4', '5.5', '5.6', '7.0', '7.1', '7.2', '7.3', '7.4', '8.0', '8.1', '8.2']
25+
php: ['5.4', '5.5', '5.6', '7.0', '7.1', '7.2', '7.3', '7.4', '8.0', '8.1', '8.2', '8.3', '8.4']
2626

2727
name: "Build Phar on PHP: ${{ matrix.php }}"
2828

@@ -77,7 +77,7 @@ jobs:
7777
# - custom_ini: Whether to run with specific custom ini settings to hit very specific
7878
# code conditions.
7979
matrix:
80-
php: ['5.4', '5.5', '5.6', '7.0', '7.1', '7.2', '7.3', '7.4', '8.0', '8.1', '8.2']
80+
php: ['5.4', '5.5', '5.6', '7.0', '7.1', '7.2', '7.3', '7.4', '8.0', '8.1', '8.2', '8.3', '8.4']
8181
custom_ini: [false]
8282

8383
include:
@@ -89,7 +89,7 @@ jobs:
8989

9090
name: "PHP: ${{ matrix.php }} ${{ matrix.custom_ini && ' with custom ini settings' || '' }}"
9191

92-
continue-on-error: ${{ matrix.php == '8.2' }}
92+
continue-on-error: ${{ matrix.php == '8.4' }}
9393

9494
steps:
9595
- name: Checkout code
@@ -132,11 +132,20 @@ jobs:
132132
# Install dependencies and handle caching in one go.
133133
# @link https://github.com/marketplace/actions/install-php-dependencies-with-composer
134134
- name: Install Composer dependencies
135+
if: ${{ matrix.php != '8.4' }}
135136
uses: "ramsey/composer-install@v2"
136137
with:
137138
# Bust the cache at least once a month - output format: YYYY-MM.
138139
custom-cache-suffix: $(date -u "+%Y-%m")
139140

141+
# For PHP "nightly", we install with ignore platform reqs.
142+
- name: Install Composer dependencies - with ignore platform
143+
if: ${{ matrix.php == '8.4' }}
144+
uses: "ramsey/composer-install@v2"
145+
with:
146+
composer-options: --ignore-platform-req=php
147+
custom-cache-suffix: $(date -u "+%Y-%m")
148+
140149
# Note: The code style check is run multiple times against every PHP version
141150
# as it also acts as an integration test.
142151
- name: 'PHPCS: set the path to PHP'

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ PHP_CodeSniffer
99
[![License](http://poser.pugx.org/phpcsstandards/php_codesniffer/license)](https://github.com/PHPCSStandards/PHP_CodeSniffer/blob/master/licence.txt)
1010

1111
![Minimum PHP Version](https://img.shields.io/packagist/php-v/phpcsstandards/php_codesniffer.svg?maxAge=3600)
12-
[![Tested on PHP 5.4 to 8.2](https://img.shields.io/badge/tested%20on-PHP%205.4%20|%205.5%20|%205.6%20|%207.0%20|%207.1%20|%207.2%20|%207.3%20|%207.4%20|%208.0%20|%208.1%20|%208.2-brightgreen.svg?maxAge=2419200)](https://github.com/PHPCSStandards/PHP_CodeSniffer/actions/workflows/test.yml)
12+
[![Tested on PHP 5.4 to 8.3](https://img.shields.io/badge/tested%20on-PHP%205.4%20|%205.5%20|%205.6%20|%207.0%20|%207.1%20|%207.2%20|%207.3%20|%207.4%20|%208.0%20|%208.1%20|%208.2%20|%208.3-brightgreen.svg?maxAge=2419200)](https://github.com/PHPCSStandards/PHP_CodeSniffer/actions/workflows/test.yml)
1313

1414
</div>
1515

0 commit comments

Comments
 (0)