Skip to content

Commit

Permalink
Merge pull request #30 from rokka-io/cs-fix
Browse files Browse the repository at this point in the history
latest cs fixer changes
  • Loading branch information
dbu authored Jan 9, 2024
2 parents fa426a5 + b5b847e commit 13f1222
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
3 changes: 3 additions & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,9 @@ jobs:
- php: '8.2'
env:
SYMFONY_REQUIRE: "*"
- php: '8.3'
env:
SYMFONY_REQUIRE: "*"
- php: '7.2'
env:
COMPOSER_FLAGS: "--prefer-lowest"
Expand Down
2 changes: 1 addition & 1 deletion src/DependencyInjection/RokkaClientExtension.php
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ public function load(array $configs, ContainerBuilder $container): void
// set the command name prefix argument for rokka commands
// we need to loop over all services of this bundle, if we use findTaggedServiceIds, things explode later inside Symfony.
foreach ($container->getDefinitions() as $name => $definition) {
if (0 === strpos($name, 'rokka.command.') && $definition->hasTag('console.command')) {
if (str_starts_with($name, 'rokka.command.') && $definition->hasTag('console.command')) {
$tagAttributes = $definition->getTag('console.command')[0];
if (\array_key_exists('command', $tagAttributes)) {
if (interface_exists(CommandLoaderInterface::class)) { // since symfony 3.4
Expand Down

0 comments on commit 13f1222

Please sign in to comment.