Skip to content

[FrameworkBundle][Validator] Remove remaining deprecations #51455

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Aug 23, 2023
Merged

[FrameworkBundle][Validator] Remove remaining deprecations #51455

merged 1 commit into from
Aug 23, 2023

Conversation

alexandre-daubois
Copy link
Member

@alexandre-daubois alexandre-daubois commented Aug 22, 2023 •
edited
Loading

Q A
Branch? 7.0
Bug fix? no
New feature? no
Deprecations? no
Tickets -
License MIT
Doc PR symfony/symfony-docs#18781

Follow-up of #51425

@carsonbot carsonbot added this to the 7.0 milestone Aug 22, 2023
@alexandre-daubois alexandre-daubois force-pushed the validator-annotations-cleanup branch 2 times, most recently from 7487530 to e4a8ea1 Compare August 22, 2023 14:57
@alexandre-daubois alexandre-daubois force-pushed the validator-annotations-cleanup branch 3 times, most recently from 0e22be6 to d00e0aa Compare August 22, 2023 17:01
@alexandre-daubois alexandre-daubois mentioned this pull request Aug 23, 2023
Merged
@alexandre-daubois alexandre-daubois force-pushed the validator-annotations-cleanup branch 2 times, most recently from 5febdec to e1ca3b5 Compare August 23, 2023 11:02
nicolas-grekas
nicolas-grekas reviewed Aug 23, 2023
View reviewed changes
@alexandre-daubois alexandre-daubois force-pushed the validator-annotations-cleanup branch 4 times, most recently from a847b54 to 9569ddd Compare August 23, 2023 11:32
derrabus
derrabus requested changes Aug 23, 2023
View reviewed changes
@alexandre-daubois alexandre-daubois force-pushed the validator-annotations-cleanup branch from 9569ddd to 6e8cab7 Compare August 23, 2023 11:47
@alexandre-daubois alexandre-daubois mentioned this pull request Aug 23, 2023
Merged
nicolas-grekas added a commit that referenced this pull request Aug 23, 2023
@nicolas-grekas
minor #51465 [Serializer] Remove @SerializedPath annotation from …
609ec6b 
…test (alexandre-daubois)

This PR was merged into the 6.4 branch.

Discussion
----------

[Serializer] Remove ``@SerializedPath`` annotation from test

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

After #51455 (comment)

Commits
-------

e5f0bd9 [Serializer] Remove ``@SerializedPath`` annotation from test
@nicolas-grekas
Copy link
Member

Thank you @alexandre-daubois.

@nicolas-grekas nicolas-grekas merged commit 056c66e into symfony:7.0 Aug 23, 2023
@alexandre-daubois alexandre-daubois deleted the validator-annotations-cleanup branch August 24, 2023 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nicolas-grekas nicolas-grekas nicolas-grekas left review comments

@derrabus derrabus derrabus requested changes

@chalasr chalasr Awaiting requested review from chalasr

@dunglas dunglas Awaiting requested review from dunglas dunglas is a code owner

Assignees
No one assigned
Labels
FrameworkBundle Status: Needs Work Validator
Projects
None yet
Milestone
7.0
Development

Successfully merging this pull request may close these issues.
4 participants
@alexandre-daubois @nicolas-grekas @derrabus @carsonbot