Skip to content

Commit

Permalink
qa: update psalm to latest 5.x commit
Browse files Browse the repository at this point in the history
Signed-off-by: Maximilian Bösing <[email protected]>
  • Loading branch information
boesing committed Oct 10, 2023
1 parent fe416be commit 6194496
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 6 deletions.
10 changes: 5 additions & 5 deletions composer.lock

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

7 changes: 6 additions & 1 deletion src/Command/AheadOfTimeFactoryCreatorCommand.php
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,6 @@ protected function execute(InputInterface $input, OutputInterface $output): int
preg_replace('/\W/', '', $factory->containerConfigurationKey)
);

/** @var class-string<FactoryInterface> $factoryClassName */
$factoryClassName = sprintf('%sFactory', $factory->fullyQualifiedClassName);
if (class_exists($factoryClassName)) {
$output->writeln(sprintf(
Expand Down Expand Up @@ -134,6 +133,12 @@ protected function execute(InputInterface $input, OutputInterface $output): int
$containerConfigurations[$factory->containerConfigurationKey] = ['factories' => []];
}

/**

Check failure on line 136 in src/Command/AheadOfTimeFactoryCreatorCommand.php

View workflow job for this annotation

GitHub Actions / ci / QA Checks (PHPCodeSniffer [8.1, locked], ubuntu-latest, laminas/laminas-continuous-integration-ac...

No assignment to $factoryClassName variable before or after the documentation comment.
* Psalm has to understand that the `factoryClassName` variable contains a class-string to a factory which
* will be available once persisted to the filesystem and loaded via composer autoloading.
*
* @var class-string<FactoryInterface> $factoryClassName
*/
$containerConfigurations[$factory->containerConfigurationKey]['factories'] += [
$factory->fullyQualifiedClassName => $factoryClassName,
];
Expand Down

0 comments on commit 6194496

Please sign in to comment.