Skip to content

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

Open
bluehaoran wants to merge 11 commits into
nategood:masterfrom
bluehaoran:PR
Open

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

Conversation

@bluehaoran

Copy link
Copy Markdown

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
Copy Markdown
Author

CI fails :/

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

@nategood

Copy link
Copy Markdown
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