Márcio Almada
Márcio Almada
:+1: this needs to be merged
Hi, @doncem! The overall idea looks fine to me. Do you need any help?
Ok. I'll try to add phpunit as a dev dependency ASAP there is time, so we won't depend on Travis environment anymore.
Doctrine annotations does that, but it comes to the expense of parsing the file containing the dockblock looking for `use` statements. An alternative approach could be to set the prefix...
Consider `MyPackage\Validation\Email` and `OtherPackage\Validation\Email` implementing a common `Yada\ValidatorInterface`, you won't be able to resolve both concrete annotations if needed.
Hi! That was actually intentional from the very beginning to consider, by default, that repeated annotations form an array of values and arrays are merged if identified with the same...
On v4 this could be adopted as a default behavior. I agree that it would be more intuitive and less ambiguous.
It will depend on how much change it would require on public classes.
For v3 it would be ok as long as it's not a huge BC break on the AnnotationsBag public API, which I believe is perfectly possible to do with internal...
Me too. This same procedure was successful last month, btw.