Skip to content

[PhpUnitBridge] Fix language deprecations incorrectly marked as direct #48122

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

Merged
merged 1 commit into from
Nov 11, 2022

Conversation

wouterj
Copy link
Member

@wouterj wouterj commented Nov 6, 2022

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

When using a vendor class that has a language deprecation (e.g. missing __serialize methods), these deprecations were incorrectly considered "direct".

@carsonbot
Copy link

Hey!

Thanks for your PR. You are targeting branch "6.2" but it seems your PR description refers to branch "4.4".
Could you update the PR description or change target branch? This helps core maintainers a lot.

Cheers!

Carsonbot

@wouterj wouterj changed the base branch from 6.2 to 4.4 November 6, 2022 12:20
@wouterj wouterj modified the milestones: 6.2, 4.4 Nov 6, 2022
@wouterj wouterj force-pushed the fix-language-deprecation-types branch 2 times, most recently from c874e8e to dfac00a Compare November 6, 2022 12:58
@carsonbot
Copy link

Hey!

I think @michaelKaefer has recently worked with this code. Maybe they can help review this?

Cheers!

Carsonbot

@wouterj wouterj force-pushed the fix-language-deprecation-types branch from dfac00a to 481a8c7 Compare November 8, 2022 14:40
@fabpot
Copy link
Member

fabpot commented Nov 11, 2022

Thank you @wouterj.

@fabpot fabpot merged commit 2e3ffe2 into symfony:4.4 Nov 11, 2022
@wouterj wouterj deleted the fix-language-deprecation-types branch November 11, 2022 07:10
@fabpot fabpot mentioned this pull request Nov 19, 2022
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.

4 participants