Skip to content

Commit

Permalink
Merge pull request #620 from CPS-IT/renovate/php-8.x
Browse files Browse the repository at this point in the history
  • Loading branch information
eliashaeussler authored Nov 25, 2024
2 parents 4b955d5 + 6939709 commit 4d9097b
Show file tree
Hide file tree
Showing 8 changed files with 9 additions and 12 deletions.
2 changes: 1 addition & 1 deletion .ddev/config.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
name: frontend-asset-handler
type: php
docroot: ""
php_version: "8.1"
php_version: "8.4"
webserver_type: nginx-fpm
router_http_port: "80"
router_https_port: "443"
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/cgl.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ jobs:
- name: Setup PHP
uses: shivammathur/setup-php@v2
with:
# @todo Use PHP 8.4 once PHP-CS-Fixer supports PHP 8.4
php-version: 8.3
tools: composer:v2, composer-require-checker, composer-unused, cs2pr
coverage: none
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ jobs:
strategy:
fail-fast: false
matrix:
php-version: ["8.1", "8.2", "8.3"]
php-version: ["8.1", "8.2", "8.3", "8.4"]
dependencies: ["locked", "highest", "lowest"]
steps:
- uses: actions/checkout@v4
Expand Down Expand Up @@ -51,7 +51,7 @@ jobs:
- name: Setup PHP
uses: shivammathur/setup-php@v2
with:
php-version: 8.3
php-version: 8.4
tools: composer:v2
coverage: pcov

Expand Down
2 changes: 1 addition & 1 deletion composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
}
],
"require": {
"php": "~8.1.0 || ~8.2.0 || ~8.3.0",
"php": "~8.1.0 || ~8.2.0 || ~8.3.0 || ~8.4.0",
"ext-filter": "*",
"ext-json": "*",
"ext-mbstring": "*",
Expand Down
4 changes: 2 additions & 2 deletions composer.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 0 additions & 4 deletions rector.php
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,6 @@
*/

use Rector\Config\RectorConfig;
use Rector\Php73\Rector\FuncCall\JsonThrowOnErrorRector;
use Rector\Php74\Rector\LNumber\AddLiteralSeparatorToNumberRector;
use Rector\Php81\Rector\FuncCall\NullToStrictStringFuncCallArgRector;
use Rector\Set\ValueObject\LevelSetList;

Expand All @@ -34,8 +32,6 @@
]);

$rectorConfig->skip([
AddLiteralSeparatorToNumberRector::class,
JsonThrowOnErrorRector::class,
NullToStrictStringFuncCallArgRector::class => [
__DIR__.'/src/Command/FetchAssetsCommand.php',
__DIR__.'/src/Command/InspectAssetsCommand.php',
Expand Down
2 changes: 1 addition & 1 deletion src/Command/ConfigAssetsCommand.php
Original file line number Diff line number Diff line change
Expand Up @@ -286,7 +286,7 @@ private function doWrite(Config\Config $config, string $path, mixed $newValue):
*/
private function buildAndValidatePath(array $assetDefinitions, string $path): string
{
$pathSegments = str_getcsv($path, '/');
$pathSegments = str_getcsv($path, '/', escape: '\\');
$strictPath = is_numeric($pathSegments[0]);

if (!$strictPath) {
Expand Down
2 changes: 1 addition & 1 deletion src/Helper/ArrayHelper.php
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@ public static function setArrayValueByPath(array $array, string $path, mixed $va
$node = &$array;

// Assure required structure in array
foreach (str_getcsv($path, '/') as $segment) {
foreach (str_getcsv($path, '/', escape: '\\') as $segment) {
if (!isset($node[$segment])) {
$node[$segment] = [];
}
Expand Down

0 comments on commit 4d9097b

Please sign in to comment.