Skip to content

we don't support php 5.4 #2

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

Merged
merged 2 commits into from
Nov 10, 2015
Merged

we don't support php 5.4 #2

merged 2 commits into from
Nov 10, 2015

Conversation

dbu
Copy link
Contributor

@dbu dbu commented Nov 10, 2015

i activated travis for this repository and noticed it tries to build with php 5.4, then composer notices that this wont work.

i suggest we activate php 7 as well. its about to be released, and currently green.

@dbu
Copy link
Contributor Author

dbu commented Nov 10, 2015

hm, but hhvm setup seems to be broken

@sagikazarmark
Copy link
Member

HHVM is an ugly bitch IMHO when it comes to cURL. PHP 7 can be enabled for this repo, but not for others, where PHPSpec is used, as 2.4 is not stable yet.

@mekras
Copy link
Collaborator

mekras commented Nov 10, 2015

I have never dealt with HHVM. So I just copied existed setup from boilerplate.

mekras added a commit that referenced this pull request Nov 10, 2015
@mekras mekras merged commit cfe07ee into master Nov 10, 2015
@sagikazarmark sagikazarmark deleted the travis-setup branch November 10, 2015 15:36
@dbu
Copy link
Contributor Author

dbu commented Nov 10, 2015 via email

@sagikazarmark
Copy link
Member

As I can see, there is a problem with the integration test suite.

@dbu
Copy link
Contributor Author

dbu commented Nov 10, 2015 via email

@mekras
Copy link
Collaborator

mekras commented Nov 11, 2015

Code coverage fixed.

@mekras
Copy link
Collaborator

mekras commented Nov 11, 2015

Tests currently needs PHP builtin server. So they can not run under HHVM. Add skipping them for now.

@mekras mekras mentioned this pull request Nov 11, 2015
13 tasks
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