Skip to content

Commit

Permalink
travis: set osrm timeout to 60s
Browse files Browse the repository at this point in the history
  • Loading branch information
@emiltin
Emil Tin authored and emiltin committed Mar 31, 2014
1 parent 2d599a5 commit 717787f
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .travis.yml
View file
Expand Up @@ -29,8 +29,8 @@ cache:
- bundler
- apt
env:
- CMAKEOPTIONS="-DCMAKE_BUILD_TYPE=Release" OSRM_PORT=5000
- CMAKEOPTIONS="-DCMAKE_BUILD_TYPE=Debug" OSRM_PORT=5010
- CMAKEOPTIONS="-DCMAKE_BUILD_TYPE=Release" OSRM_PORT=5000 OSRM_TIMEOUT=60
- CMAKEOPTIONS="-DCMAKE_BUILD_TYPE=Debug" OSRM_PORT=5010 OSRM_TIMEOUT=60
notifications:
irc:
channels:
Expand Down

5 comments on commit 717787f

@emiltin
Copy link
Contributor

@emiltin emiltin commented on 717787f Apr 1, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

after increasing the timeout, no scenarios have yet failed on travis

@DennisOSRM
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Again, refrain from pushing to develop directly. This should have gone into PR #970.

@emiltin
Copy link
Contributor

@emiltin emiltin commented on 717787f Apr 1, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

alright.
it's still ok for me to push to directly to develop when it's only related to cucumber right? i thought these commits fell under that category. but of course i'm happy to use more PR's if you like.

@DennisOSRM
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure, the things under /test, /features, etc are fine. It just would have fit the PR quite nicely 😀

@emiltin
Copy link
Contributor

@emiltin emiltin commented on 717787f Apr 1, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

alright. these particular commits was in fact also in the PR :-)

Please sign in to comment.