Skip to content
This repository was archived by the owner on May 1, 2019. It is now read-only.

Conversation

ins0
Copy link
Contributor

@ins0 ins0 commented Oct 27, 2015

I think we all agree that the Travis HHVM runs don't accomplish anything and only use resources that we really don't need.

This PR

  • removes Travis HHVM runs
  • adds PHP7 Travis runs

@ins0 ins0 changed the title Enhancement/Removed hhvm runs and added php7 runs Enhancement/Removed hhvm runs Oct 27, 2015
@Ocramius
Copy link
Member

@ins0 why did you remove 7?

@ins0
Copy link
Contributor Author

ins0 commented Oct 27, 2015

@Ocramius the php7 runs allways fails, as of the required composer dependencies. Should it stay?

@Ocramius
Copy link
Member

I think it should, as we will move to it when possible :|

@ins0 ins0 force-pushed the enhancement/php7 branch from 237c360 to b87f658 Compare October 27, 2015 12:36
@ins0
Copy link
Contributor Author

ins0 commented Oct 27, 2015

@Ocramius ok i was on the "Don't waste Travis Resources" Plan, added 7 support :)

@Ocramius
Copy link
Member

@ins0 thanks!

Ocramius added a commit that referenced this pull request Oct 27, 2015
Enhancement/Removed hhvm runs
@Ocramius Ocramius merged commit 6e0e785 into zendframework:master Oct 27, 2015
@Ocramius Ocramius self-assigned this Oct 27, 2015
@ins0
Copy link
Contributor Author

ins0 commented Oct 27, 2015

thanks @Ocramius

@ins0 ins0 deleted the enhancement/php7 branch October 27, 2015 12:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants