Skip to content

Conversation

@gassan
Copy link
Contributor

@gassan gassan commented Jan 16, 2022

subj.
Plase adjuct .scrutinizer.yml config if necessary.

Symfony\Bridge\PhpUnit\SymfonyTestsListener added to output deprecations during tests
Copy link

@vgpastor vgpastor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why delete phpstan?
phpfixer and phpstan are compatibles and works fine together

docker-compose run --rm -T php /usr/local/bin/php /app/vendor/bin/phpunit

phpstan: clear-cache
docker-compose run --rm -T php /usr/local/bin/php /app/vendor/bin/phpstan analyse

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why delete phpstan?
phpfixer and phpstan are compatibles and works fine together

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very strange. I think it was a mistake.

@gassan gassan force-pushed the php-cs-fixer-added branch 2 times, most recently from b9521d5 to 96440b9 Compare January 30, 2023 09:50
@gassan gassan force-pushed the php-cs-fixer-added branch from 96440b9 to 9a2266a Compare January 30, 2023 09:52
@gassan gassan changed the title Php cs fixer added Php cs fixer added and fixed SF6 deprecation notice Jan 30, 2023
@roboticflamingo
Copy link

Can this be merged? the deprecation notice is really annoying on 8.2! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants