Skip to content

Commit a55ecd3

Browse files
committed
fix: Avoid implicit null argument for php 8.4
1 parent b3cb510 commit a55ecd3

File tree

3 files changed

+5
-5
lines changed

3 files changed

+5
-5
lines changed

.github/workflows/validate-code.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ jobs:
3737
strategy:
3838
fail-fast: true
3939
matrix:
40-
php: [ "8.1" ]
40+
php: [ "8.1", "8.2", "8.3", "8.4" ]
4141
os: [ "ubuntu-latest" ]
4242
steps:
4343
- name: Checkout

src/Parser/PHP/Strategy/AnnotationStrategy.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ final class AnnotationStrategy implements StrategyInterface
3737
public function __construct(
3838
ConstExprParser $constExprParser,
3939
Lexer $lexer,
40-
ParserConfig $parserConfig = null
40+
?ParserConfig $parserConfig = null
4141
) {
4242
$this->lexer = $lexer;
4343
if ($parserConfig !== null) {

tests/Integration/AbstractIntegrationTestCase.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -50,8 +50,8 @@ private function loadVendorPackage(string $cwd, string $packageName): PackageInt
5050
*/
5151
protected function loadDefinedFileSymbols(
5252
string $baseDir,
53-
array $autoloadTypes = null,
54-
string $packageName = null
53+
?array $autoloadTypes = null,
54+
?string $packageName = null
5555
): array {
5656
if ($packageName === null) {
5757
$package = $this->loadPackage($baseDir);
@@ -86,7 +86,7 @@ protected function loadDefinedFileSymbols(
8686
* @return array<SymbolInterface>
8787
* @param list<string> $excludedDirs
8888
*/
89-
protected function loadConsumedFileSymbols(string $baseDir, array $autoloadTypes = null, array $excludedDirs = []): array
89+
protected function loadConsumedFileSymbols(string $baseDir, ?array $autoloadTypes = null, array $excludedDirs = []): array
9090
{
9191
$rootPackage = $this->loadPackage($baseDir);
9292

0 commit comments

Comments
 (0)