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

Declare compatibility with PHP 8.x #19

Merged
merged 1 commit into from
Nov 2, 2020
Merged

Conversation

m-ober
Copy link
Contributor

@m-ober m-ober commented Oct 10, 2020

I don't see why this package should not be declared as compatible with PHP 8.

@edudobay edudobay mentioned this pull request Oct 10, 2020
1 task
@m-ober
Copy link
Contributor Author

m-ober commented Oct 12, 2020

Maybe it would even be possible to make the version unconstrained, so composer installs for new PHP versions are not blocked by this package.

Copy link
Member

@Jean85 Jean85 left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Uncostrained is dangerous IMHO for such long-lived versions like PHP-FIG's packages, since we could risk getting those installed even if not compatible at all in the future. ^8.0 is enough.

@codemasher
Copy link

Hi, a review and merge would be greatly appreciated because this package is currently breaking PHP8 builds. Thanks!

@edudobay edudobay mentioned this pull request Nov 1, 2020
5 tasks
@weierophinney weierophinney merged commit ff4661e into php-fig:master Nov 2, 2020
@edudobay
Copy link

Hi! A tagged release would be greatly appreciated so other packages don't need to depend on dev-master. Are there plans for that with the nearing PHP 8.0 GA release? Thanks!

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