Alessandro Lai
Alessandro Lai
@mbabker I addressed your comments with some reverts; it may be a good idea to squash this PR while merging.
Thanks @mbabker, I've addressed all the comments of your review.
Ok I've debugged the issue, that lies in `\Doctrine\ORM\Mapping\Driver\ReflectionBasedDriver::isRepeatedPropertyDeclaration`: * that method should return `true`, skipping the whole "double-mapping" * ...but it doesn't because: * the reflection says that the...
Bisecting and using this as the reproducer, I've discovered that my chain of inheritance was considered "bad" but accepted in 2.x only thanks to the `reportFieldsWhereDeclared` set to `false`. Should...
Locally, the schema validator is what led me to this reproducer, but it's an exception, not a validation error. This reproducer, ported to the 2.x branch, fails if `reportFieldsWhereDeclared` is...
PHPUnit 11 is not getting installed, probably because some other dependency conflicts with it. I will not merge for now.
Note: `getModeString()` is present since 1.7.0, so this would need a slight bump to the `ext-mongodb` requirement: https://www.php.net/manual/en/mongodb-driver-readpreference.getmodestring.php
@sebastianbergmann unfortunately I just hit this same bug, because I run my tests with a "Docker in Docker" approach, so the absolute path does not match what GitLab expects, hence...
@dhirtzbruch can you tell us how that workaround worked for you, or with which package versions? I rolled back to PHPUnit 9.4.0 and php-code-coverage 9.2.0 (the first versions to introduce...
I think I found the issue by debugging it: it's not in the Cobertura implementation but one step before, in `\SebastianBergmann\CodeCoverage\Node\Builder::reducePaths`: since all my covered code is inside `src/`, that...