Added Travis-CI #88

Closed
wants to merge 7 commits into
from

Projects

None yet

4 participants

lenada commented Jan 25, 2013

Thought it would be nice to have travis.
testReferenceReconstruction is failing at the moment

@stloyd stloyd and 1 other commented on an outdated diff Jan 25, 2013
.travis.yml
@@ -0,0 +1,14 @@
+language: php
+
+php:
+ - 5.3
+ - 5.4
+
+before_script:
+ - wget http://getcomposer.org/composer.phar -O ./composer.phar
stloyd
stloyd Jan 25, 2013

Composer is available at Travis-CI. Use simply composer install --dev and use --prefer-source as this allows to skip GH limits.

lenada
lenada Jan 26, 2013

good point to use the preinstalled composer on travis (although it has not been updated for a while)
I have also run into GH limits with using --prefer-source. So does it really make a difference compared to --prefer-dist, limit-wise ?

@lenada lenada use travis composer and prefer source to not run into GH limit
do what @stloyd says:
"Composer is available at Travis-CI. Use simply composer install --dev and use --prefer-source as this allows to skip GH limits."
8a94b71
@stloyd stloyd commented on an outdated diff Jan 25, 2013
.travis.yml
@@ -0,0 +1,12 @@
+language: php
+
+php:
+ - 5.3
+ - 5.4
+
+before_script:
+ - composer install --prefer-source --dev
+ - gem install colored -v=1.2
+ - cp phpunit.xml.dist phpunit.xml
stloyd
stloyd Jan 25, 2013

IIRC there is no need for this, as from some version, phpunit by default looks for phpunit.xml.dist file.

@lenada lenada remove cp phpunit.xml.dist
as recent phpunit versions check automatically for phpunit.xml.dist
819a088
@stof stof and 1 other commented on an outdated diff Jan 26, 2013
.travis.yml
@@ -0,0 +1,11 @@
+language: php
+
+php:
+ - 5.3
+ - 5.4
+
+before_script:
+ - composer install --prefer-source --dev
+ - gem install colored -v=1.2
stof
stof Jan 26, 2013 Member

why this gem ?

lenada
lenada Jan 26, 2013

yeah good question ;) it does not make any sense

Member

Please rebase your PR.

Member

testReferenceReconstruction error will be fixed by #177

lenada commented Mar 22, 2015

@lavoiesl I think this PR can be closed, as travis-ci support has already landed in master 69c2230

@lenada lenada closed this Mar 22, 2015
@lenada lenada deleted the unknown repository branch Mar 22, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment