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

Test against php 8.0 #520

Merged
merged 1 commit into from
Mar 7, 2022
Merged

Test against php 8.0 #520

merged 1 commit into from
Mar 7, 2022

Conversation

sergiy-petrov
Copy link
Contributor

No description provided.

@matthewtrask
Copy link
Contributor

it looks like the tests for php 8 have failed, can you add 8 to the allowed-failures for the time being until we get time to update the package?

@sergiy-petrov
Copy link
Contributor Author

@matthewtrask i tried to resolve this issue by requiring phpunit ^9.3 in composer.json but it seems it's impossible to have php 5.4 and 8.0 support at the same time (because phpunit ^9.3 forces you to use :void typehint in setUp() method, but typehints are not supported in older versions of php).

What do you think about dropping legacy php version in the next major version of the package? I can PR that

@willishq
Copy link
Member

willishq commented May 5, 2021

I fully advocate the removal of legacy versions of PHP from our supported versions, PHP 7.2 is currently the latest version to reach EOL. PHP 5.4 has not been supported for almost 6 years. Realistically I think the minumum version requirement for the next major version should be 7.4.

@JustSteveKing
Copy link
Contributor

I think by the time a new major version is released, PHP will be phasing out 7.4?

Copy link
Member

@Nyholm Nyholm left a comment

Choose a reason for hiding this comment

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

I suggest to merge and fix the issues in the following PR.

Related to #533

@Nyholm Nyholm merged commit ebc071a into thephpleague:master Mar 7, 2022
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.

5 participants