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

Package php-http/message-factory is abandoned, you should avoid using it. Use psr/http-factory instead #433

Closed
zerkms opened this issue Jun 12, 2023 · 7 comments · Fixed by #439

Comments

@zerkms
Copy link

zerkms commented Jun 12, 2023

PHP version: x.y.z (hint: php --version) 8.1.9

Description

At the moment HttplugBundle depends on php-http/message-factory

"php-http/message-factory": "^1.0.2",

which was deprecated.

How to reproduce

Possible Solution

Additional context

@ruudk
Copy link
Contributor

ruudk commented Aug 25, 2023

@dbu I looked into this, but how is this supposed to be handled for this bundle? I think it can only be done in v2?

@dbu
Copy link
Collaborator

dbu commented Aug 25, 2023

yep, we can't avoid this without a BC break.

we did prepare in #427 , exposing the psr stuff for autowiring.

i don't have much time for OS atm, but would not be oposed to people looking into a new major version and getting rid of legacy things. the current version seems really stable - it is used in quite some places and there are very little complaints. so making 2.0 should not impact 1.x users too much as not much is going on anyways.

we could weed out quite a bit of the DI configuration code, and also simplify the documentation because we don't need to explain that much deprecated stuff anymore.

@dbu
Copy link
Collaborator

dbu commented Nov 24, 2023

fixed in #439, will be in version 2.

@dbu dbu closed this as completed Nov 24, 2023
@Prometee
Copy link
Contributor

Hello @dbu do you have a date to release 2.0 ? It looks like it's pretty stable, do you need help on something to release it ?

@dbu
Copy link
Collaborator

dbu commented Jun 10, 2024

hi, yeah we should release soon. the one thing missing is cleaning up the code for strict typing, as that is a BC break when people extend classes. and #321 as well.

if you have some time to check that all class properties, arguments and methods are typed, that would certainly help. if you could do a pull request with it, all the better.

if not i will try to find time for it later this week.

@Prometee
Copy link
Contributor

Prometee commented Jun 10, 2024

What about adding PHPStan and Easy Coding Standard (it use PHP-CS-Fixer) to the CI ?
It will help keeping some basic rules for this repository and I think it can take care of some types requirements.

I will start creating the PR you mentioned and then see if PHPStan and ECS can be added and have any benefits.

EDIT: I didn't notice that PHP CS fixer was already used so no need for ECS, then PHPStan can be the way to detect issues with non typed properties.

@dbu
Copy link
Collaborator

dbu commented Jun 11, 2024

awesome if you have time to add phpstan and fix easy issues / generate a baseline for the rest. we have it set up in https://github.com/php-http/httplug/ - if you can do it similar that will help with consistency across the repositories. 👍

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 a pull request may close this issue.

4 participants