Skip to content

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
Collaborator

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
Collaborator

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
Collaborator

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
Collaborator

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

3 participants