Skip to content
This repository has been archived by the owner on Jun 27, 2022. It is now read-only.

Update .travis.yml #34

Closed
wants to merge 1 commit into from
Closed

Update .travis.yml #34

wants to merge 1 commit into from

Conversation

GrahamCampbell
Copy link
Member

There's no point in testing on php 5.6 and hhvm when it clearly doesn't work. Also, I've fixed the composer install issues where it was downloading everything twice because dist install was failing.

@mtdowling
Copy link
Member

I don't want to remove them from the tests. It's there so that the issues with those versions are known and can be fixed.

Also, I've fixed the composer install issues where it was downloading everything twice because dist install was failing.

Can you update the PR to just be this change?

@GrahamCampbell
Copy link
Member Author

What I was meaning is we that we don't even get to the point of running the test suite on those versions, so why waste travis resources? Surely it would be a better idea to only enable those versions once that's actually been fixes?

@GrahamCampbell GrahamCampbell deleted the travis branch August 6, 2014 21:01
@mtdowling
Copy link
Member

Instead of removing them, I'd like to just keep this issue open to track it.

  1. Testing pecl-http on 5.6 fails. We could just remove the pecl tests from Travis. It's only needed for testing a single fringe class that is probably not utilized by anyone.
  2. Calling pear on HHVM: The command "pear config-set php_ini ~/.phpenv/versions/php -r 'echo phpversion();'/etc/php.ini" failed and exited with 127 during .. This is also due to having to test pecl-http I believe.

@GrahamCampbell GrahamCampbell restored the travis branch August 6, 2014 21:19
@GrahamCampbell GrahamCampbell reopened this Aug 6, 2014
@mtdowling
Copy link
Member

HHVM and 5.6 are now fixed.

@mtdowling mtdowling closed this Aug 11, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants