- Issue created by @bbrala
- Merge request !11940feaT: add SequentiallyConstraint and childs and test if we can have our... → (Open) created by bbrala
- 🇳🇱Netherlands bbrala Netherlands
Wow, am i dreaming. This little trick might make it possible to support those symfony constraints without weirdness.
Everything is green... It cant be that easy right?
- 🇺🇸United States smustgrave
Left some small comments and a question
If you are another contributor eager to jump in, please allow the previous poster(s) at least 48 hours to respond to feedback first, so they have the opportunity to finish what they started!
- 🇳🇱Netherlands bbrala Netherlands
Fixed issues, but kinda need someone into validation to review this since it feels weird this works at face value. It might overlook issues.
- 🇳🇱Netherlands bbrala Netherlands
Removing tag since committer who will review this seems to be the subsystem maintainer.
- 🇺🇸United States smustgrave
Going to mark because I believe all feedback has been addressed on this one, thank you for adding the CR.
- 🇳🇿New Zealand quietone
I did not do a code review but I left a question about the name of the constraint and then made some minor suggestions.
This should have another opinion about the name, not just mine.