Skip to content

Conversation

alexandre-daubois
Copy link
Member

@alexandre-daubois alexandre-daubois commented Aug 22, 2024

Fix #20148

May be tweaked a bit if symfony/symfony#58064 passes.

@carsonbot carsonbot added this to the 7.2 milestone Aug 22, 2024
nicolas-grekas added a commit to symfony/symfony that referenced this pull request Aug 22, 2024
…Case` (alexandre-daubois)

This PR was merged into the 7.2 branch.

Discussion
----------

[Validator] Add ``@template`` on `CompoundConstraintTestCase`

| Q             | A
| ------------- | ---
| Branch?       | 7.2
| Bug fix?      | no
| New feature?  | yes
| Deprecations? | no
| Issues        | -
| License       | MIT

While writing [the docs](symfony/symfony-docs#20150), I realized ``@template`` could be added. Moreover, it is already present on `ConstraintValidatorTestCase`.

Commits
-------

618215a [Validator] Add ``@template`` on CompoundConstraintTestCase`
@xabbuh
Copy link
Member

xabbuh commented Aug 23, 2024

@alexandre-daubois symfony/symfony#58064 has been merged

@alexandre-daubois
Copy link
Member Author

Thanks! Updated

@javiereguiluz javiereguiluz merged commit 84a9a57 into symfony:7.2 Aug 26, 2024
3 checks passed
@javiereguiluz
Copy link
Member

Alex, another awesome contribution! You're on a roll! Thanks 🙇

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants