search

doctrine / DoctrineMongoDBBundle

Integrates Doctrine MongoDB ODM with Symfony
http://symfony.com/doc/current/bundles/DoctrineMongoDBBundle/index.html
MIT License
377 stars 229 forks source link

Bump coding standard to v11 #756

Closed franmomu closed 1 year ago

franmomu commented 1 year ago

Use the latest doctrine/coding-standard version (it supports "php": "^7.2 || ^8.0")

IonBazan commented 1 year ago

@franmomu please rebase - there is a conflict after I merged #755

franmomu commented 1 year ago

I have to check why it fails 😕

franmomu commented 1 year ago

I guess it's because in:

https://github.com/symfony/symfony/blob/2f62c4a7951ed05e722638beb3dcdea7c650a3a2/src/Symfony/Bridge/Doctrine/DependencyInjection/AbstractDoctrineExtension.php#L320-L326

it doesn't match with /** @Document **/, I guess there should be a way to not modify these annotation for now.

IonBazan commented 1 year ago

Ouch, yes. I think you can add all test fixtures to ignore the coding standard or at least that particular rule.

IonBazan commented 1 year ago

Thanks @franmomu!