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

use a PHPUnit bridge version that is compatible with recent PHPUnit releases #25

Closed
wants to merge 1 commit into from

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Dec 4, 2023

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Related tickets
Documentation
License MIT

@xabbuh xabbuh changed the title use a PHPUnit bridge version that is compatible with recent PHPUnit r… use a PHPUnit bridge version that is compatible with recent PHPUnit releases Dec 4, 2023
Copy link
Contributor

@dbu dbu left a comment

Choose a reason for hiding this comment

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

thanks for looking into this.

the phpstan error is our friend from php-http/client-common#234 (comment) - not related to your changes.

can you maybe check the flex composer plugin error? maybe we just never built this package since composer started having configuration to control plugin execution and we need to allow the flex plugin.

composer.json Outdated Show resolved Hide resolved
@KDederichs
Copy link
Contributor

I fixed the Flex error in my PR, if you merge this I can merge it into mine and it should be good I think

@dbu
Copy link
Contributor

dbu commented Dec 5, 2023

thank you!

merged in #24

@dbu dbu closed this Dec 5, 2023
@xabbuh xabbuh deleted the phpunit-bridge branch December 5, 2023 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants