Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Validator] deprecate the use of option arrays to configure validation constraints #54744

Open
wants to merge 1 commit into
base: 7.1
Choose a base branch
from

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Apr 26, 2024

Q A
Branch? 7.1
Bug fix? no
New feature? no
Deprecations? yes
Issues Fix #51393
License MIT

@carsonbot carsonbot added this to the 7.1 milestone Apr 26, 2024
@xabbuh xabbuh modified the milestones: 7.1, 7.2 Apr 26, 2024
@xabbuh xabbuh force-pushed the issue-51393 branch 2 times, most recently from 0c07d99 to 0d1d920 Compare April 26, 2024 11:07
@xabbuh xabbuh requested a review from yceruto as a code owner April 26, 2024 11:07
@xabbuh xabbuh force-pushed the issue-51393 branch 5 times, most recently from 1a3ee80 to e58d5f9 Compare April 28, 2024 19:01
fabpot added a commit that referenced this pull request May 1, 2024
…s with named arguments (xabbuh)

This PR was merged into the 6.4 branch.

Discussion
----------

[Validator] handle edge cases when constructing constraints with named arguments

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

spotted while working on #54744

Commits
-------

93eb8a0 handle edge cases when constructing constraints with named arguments
@alexandre-daubois
Copy link
Contributor

What a PR 👏

Quick question: don't we have deprecations expectations tests? I was not able to find tests where the new deprecations are expected. Did I miss them or is it avoided for some reason?

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.

[RFC][Validator] Remove constraint construction from associative arrays
3 participants