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

Fix #883 with support for php 5 #916

Closed
wants to merge 3 commits into from

Conversation

silverbackdan
Copy link

This is an alternative approach to #884

I think all the methods being used here from ReflectionClass are supported from PHP 5 so hopefully tests will pass.

@alcaeus
Copy link
Member

alcaeus commented Feb 4, 2019

Thanks for the attempt @silverbackdan, but this is not going to work. I'll close this PR as it has the same flaws as the original.

@alcaeus alcaeus closed this Feb 4, 2019
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.

None yet

2 participants