Alessandro Lai

Results 270 comments of 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...