Skip to content

Allow Request to be extended by changing class name 'Request' to static#264

Open
bluehaoran wants to merge 11 commits intonategood:masterfrom
bluehaoran:PR
Open

Allow Request to be extended by changing class name 'Request' to static#264
bluehaoran wants to merge 11 commits intonategood:masterfrom
bluehaoran:PR

Conversation

@bluehaoran
Copy link

This replaces #231 because that became a bit of a tangled mess.

This is cleaner and changes just one file, and doesn't fail CI.

@bluehaoran
Copy link
Author

CI fails :/

  • Does HHVM have issues with the static keyword?
  • Is the PHP7 test failing across the board?

@nategood
Copy link
Owner

I don't believe we've tried testing either. Will add 7 to CI in a different PR.

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.

2 participants