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

Drop PHP 7.4 #1120

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Drop PHP 7.4 #1120

wants to merge 1 commit into from

Conversation

simPod
Copy link
Collaborator

@simPod simPod commented Apr 25, 2022

I was wondering that for master we could drop PHP 7.4 already.

@simPod simPod requested a review from spawnia April 25, 2022 08:55
@spawnia
Copy link
Collaborator

spawnia commented Apr 25, 2022

I would like to make the switch when its worth doing. With PHP 8, I would like to redesign quite a few APIs and use named parameters.

Let's hold off on that until the next major version, since we are currently closing in on the v15 release.

@simPod simPod added this to the 16.0.0 milestone Apr 25, 2022
@spawnia spawnia marked this pull request as draft May 31, 2022 15:22
@ruudk
Copy link
Contributor

ruudk commented May 26, 2023

V15 is released since some time and works pretty well. Would it be a good moment to start developing V16?

@spawnia
Copy link
Collaborator

spawnia commented May 28, 2023

I personally still have a need for PHP 7.4 compatibility and am not eager to drop it.

@ruudk
Copy link
Contributor

ruudk commented Aug 15, 2023

Sorry to ask, but are there any changes in your situation @spawnia? I'm currently working on #1426 and I feel quiet silly to not be able to use union types and have to fallback to no type safety at all. PHP 7.4 is EOL for a long time now. In my opinion it does not make any sense to keep supporting this.

@spawnia
Copy link
Collaborator

spawnia commented Aug 16, 2023

I still have an important project to maintain that uses this library through https://github.com/spawnia/sailor, and probably will for a couple of years. I guess GraphQL is pretty slow to change aspects that affect the client, so I might not have to worry about backwards compatibility that much.

I can definitely see the advantages of using a newer PHP version for development. Perhaps we can implement something like https://getrector.com/blog/how-all-frameworks-can-bump-to-php-81-and-you-can-use-older-php to get the best of both?

@ruudk
Copy link
Contributor

ruudk commented Aug 16, 2023

Just wondering, but why does Sailor have to keep supporting PHP 7.4?

I don't really think it's worth the effort to downgrade this project with Rector. Gives you a lot of other issues to deal with, and for what?

@spawnia
Copy link
Collaborator

spawnia commented Aug 16, 2023

I just explained why, and I am starting to get annoyed by your repeated questioning of my reasons.

@ruudk
Copy link
Contributor

ruudk commented Aug 16, 2023

Alright, I misread it. Not my intention to annoy you ☺️

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

3 participants