Skip to content

Commit

Permalink
Merge pull request #22 from homersimpsons/fix/phpstan-typing
Browse files Browse the repository at this point in the history
Fix CI: PHPStan's error, Dependency conflicts, MySQL version
  • Loading branch information
homersimpsons authored Apr 2, 2021
2 parents 4450c0f + 97aeedc commit ccdb2c3
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/continuous-integration.yml
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ jobs:

services:
mysql:
image: "mysql:8"
image: "mysql:5.7"
env:
MYSQL_ALLOW_EMPTY_PASSWORD: true
MYSQL_ROOT_PASSWORD:
Expand Down
2 changes: 1 addition & 1 deletion DependencyInjection/TdbmCompilerPass.php
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ class TdbmCompilerPass implements CompilerPassInterface
/**
* @inheritDoc
*/
public function process(ContainerBuilder $container)
public function process(ContainerBuilder $container): void
{
$generatorListeners = $this->taggedServicesToReferences($container->findTaggedServiceIds(TdbmExtension::TAG_GENERATOR_LISTENER));
$codeGeneratorListeners = $this->taggedServicesToReferences($container->findTaggedServiceIds(TdbmExtension::TAG_CODE_GENERATOR_LISTENER));
Expand Down
2 changes: 1 addition & 1 deletion TdbmBundle.php
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@

class TdbmBundle extends Bundle
{
public function build(ContainerBuilder $container)
public function build(ContainerBuilder $container): void
{
parent::build($container);

Expand Down
4 changes: 2 additions & 2 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,8 @@
"require" : {
"php" : ">=7.2",
"thecodingmachine/tdbm" : "~5.2.0",
"doctrine/doctrine-bundle": "^1.9 || ^2",
"doctrine/orm": "^1 || ^2",
"doctrine/doctrine-bundle": "^2",
"doctrine/orm": "^2",
"symfony/http-kernel": "^4.1.9 || ^5"
},
"require-dev": {
Expand Down

0 comments on commit ccdb2c3

Please sign in to comment.