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

[PhpUnitBridge] Fix ExpectDeprecationTrait::expectDeprecation() conflict #37153

Merged

Conversation

fancyweb
Copy link
Contributor

@fancyweb fancyweb commented Jun 8, 2020

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

ExpectDeprecationTrait::expectDeprecation() must be compatible with TestCase::expectDeprecation(). I'm personally against a renaming on our side because this name is the best.

@fancyweb fancyweb force-pushed the php-unit-bridge-expect-deprecation-compat branch 2 times, most recently from 8911978 to 6b9d6c0 Compare June 8, 2020 16:15
Copy link
Member

@nicolas-grekas nicolas-grekas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks, that's the approach I was about to recommend :)

@nicolas-grekas nicolas-grekas added this to the 5.1 milestone Jun 8, 2020
@fancyweb fancyweb force-pushed the php-unit-bridge-expect-deprecation-compat branch from 6b9d6c0 to 1e61bde Compare June 9, 2020 09:30
@fancyweb fancyweb force-pushed the php-unit-bridge-expect-deprecation-compat branch from 1e61bde to ea5bb24 Compare June 9, 2020 09:56
@fabpot
Copy link
Member

fabpot commented Jun 10, 2020

Thank you @fancyweb.

@fabpot fabpot merged commit 5cefcc2 into symfony:5.1 Jun 10, 2020
@fancyweb fancyweb deleted the php-unit-bridge-expect-deprecation-compat branch June 10, 2020 07:03
@fabpot fabpot mentioned this pull request Jun 12, 2020
xabbuh added a commit that referenced this pull request Sep 18, 2020
…ridge (xabbuh)

This PR was merged into the 5.1 branch.

Discussion
----------

[Validator] fix lowest allowed version for the PHPUnit bridge

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

Fixes the `deps=low` build for the Validator component by only allowing versions of the PHPUnit bridge that contain #37153.

Commits
-------

fb7ee79 fix lowest allowed version for the PHPUnit bridge
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.

None yet

5 participants