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

Make use of PHP 7 features #122

Closed
wants to merge 2 commits into from
Closed

Make use of PHP 7 features #122

wants to merge 2 commits into from

Conversation

kelunik
Copy link
Member

@kelunik kelunik commented Dec 23, 2016

Adds scalar type declarations and return type declarations. Increases the minimum version of PHP to 7.0.0.

We can discuss whether we make the error handler an interface now, as we have anonymous classes.

Relates to #2.

Adds scalar type declarations and return type declarations. Increases the minimum version of PHP to 7.0.0.
@kelunik
Copy link
Member Author

kelunik commented Dec 23, 2016

Does anybody know the current state of PHP 7 features implemented in HHVM? Do we care about HHVM?

@WyriHaximus
Copy link
Member

Does anybody know the current state of PHP 7 features implemented in HHVM? Do we care about HHVM?

HHVM doesn't expose itself to as PHP 7 to composer table flips when you try to run composer install on it. Didn't go any further and pretty much dropping HHVM from my PHP 7 only package over the last months because of that. No idea what PHP 7 features they would support

@bwoebi
Copy link
Member

bwoebi commented Dec 23, 2016

Can you please PR the fixes separately (like detatch => detach, $time => $msDelay etc.)?

@kelunik
Copy link
Member Author

kelunik commented Dec 29, 2016

Closing as of #2 (comment).

@kelunik kelunik closed this Dec 29, 2016
@kelunik kelunik deleted the php7 branch December 29, 2016 16:45
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.

3 participants