Skip to content

Commit 35098d7

Browse files
committed
Merge remote-tracking branch 'upstream/4.x' into issue-1082/exit-code-when-fixing-stdin
2 parents 87c0b23 + ceb3624 commit 35098d7

File tree

4 files changed

+9
-17
lines changed

4 files changed

+9
-17
lines changed

.github/workflows/test.yml

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -191,7 +191,7 @@ jobs:
191191
- name: Install PHP
192192
uses: shivammathur/setup-php@v2
193193
with:
194-
php-version: ${{ matrix.php == '8.5' && '8.4' || matrix.php }}
194+
php-version: ${{ matrix.php }}
195195
ini-values: ${{ steps.set_ini.outputs.PHP_INI }}
196196
extensions: ${{ matrix.extensions }}
197197
coverage: none
@@ -207,14 +207,6 @@ jobs:
207207
composer-options: ${{ matrix.php == '8.5' && '--ignore-platform-req=php+' || '' }}
208208
custom-cache-suffix: $(date -u "+%Y-%m")
209209

210-
- name: "Install PHP again (PHP 8.5)"
211-
if: ${{ matrix.php == '8.5' }}
212-
uses: shivammathur/setup-php@v2
213-
with:
214-
php-version: ${{ matrix.php }}
215-
ini-values: ${{ steps.set_ini.outputs.PHP_INI }}
216-
coverage: none
217-
218210
- name: Grab PHPUnit version
219211
id: phpunit_version
220212
shell: bash

tests/Core/Config/IniSetTest.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -300,10 +300,10 @@ public static function dataIniValueCannotBeUpdatedAtRuntime()
300300
return [
301301
// Using Core directives available PHP cross-version to prevent the tests failing
302302
// on an unavailable directive or due to an extension not being available.
303-
'php.ini only option: disable_classes' => [
304-
'option' => 'disable_classes',
305-
'newValue' => 'DateTime,DOMComment',
306-
'alternativeValue' => 'DOMComment,DateTime',
303+
'php.ini only option: expose_php' => [
304+
'option' => 'expose_php',
305+
'newValue' => '0',
306+
'alternativeValue' => '1',
307307
],
308308
'INI_PERDIR option: short_open_tag (bool)' => [
309309
'option' => 'short_open_tag',

tests/Core/Ruleset/IniSetFailIniOnlyTest.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
<ruleset xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" name="IniSetTest" xsi:noNamespaceSchemaLocation="phpcs.xsd">
33
<description>Ruleset to test ini values which can not be changed at runtime will be reported as such when set from the ruleset.</description>
44

5-
<ini name="disable_classes" value="DateTime,DOMComment"/>
5+
<ini name="expose_php" value="0"/>
66

77
<!-- Prevent a "no sniff were registered" error. -->
88
<rule ref="Generic.PHP.BacktickOperator"/>

tests/Core/Ruleset/ProcessRulesetIniSetTest.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -116,10 +116,10 @@ public static function dataIniValueCannotBeUpdatedAtRuntime()
116116
return [
117117
// Using Core directives available PHP cross-version to prevent the tests failing
118118
// on an unavailable directive or due to an extension not being available.
119-
'php.ini only option: disable_classes' => [
119+
'php.ini only option: expose_php' => [
120120
'standard' => __DIR__.'/IniSetFailIniOnlyTest.xml',
121-
'option' => 'disable_classes',
122-
'expected' => 'DateTime,DOMComment',
121+
'option' => 'expose_php',
122+
'expected' => '0',
123123
],
124124
'INI_PERDIR option: short_open_tag' => [
125125
'standard' => __DIR__.'/IniSetFailIniPerDirTest.xml',

0 commit comments

Comments
 (0)