Skip to content
This repository has been archived by the owner on Nov 13, 2017. It is now read-only.

[jade] Update Travis conf for Ubuntu Trusty. #285

Merged
merged 1 commit into from
Apr 27, 2016

Conversation

130s
Copy link
Contributor

@130s 130s commented Apr 25, 2016

Could any admins enable Travis at https://travis-ci.org/profile/ros-planning/ (can be done with a single click) and poke me? Thanks in advance.

(I assume the reason why Travis is disabled is the same as moveit/moveit_commander#44 (comment)).

@davetcoleman
Copy link
Member

poke

@130s
Copy link
Contributor Author

130s commented Apr 26, 2016

@jacquelinekay @wjwwood Could any of you help us again?
And moveit_ros repository too if possible. Just one click more ;)

@davetcoleman davetcoleman merged commit 3b7be14 into moveit:jade-devel Apr 27, 2016
@130s 130s deleted the jade/impr/travis branch April 28, 2016 06:07
@davetcoleman
Copy link
Member

@130s I've been looking at this new .travis.yml file further and have some questions:

  • Can we move all the if [[ ${PRERELEASE} == true ]]; scripts to a separate bash script in the repo to simplify reading? e.g. .rosPreRelease.sh ?
  • Or could we integrate the pre-release stuff into industrial_ci, where it belongs IMHO?
  • Why does this travis file looks so different than the one documented on industrial_ci? e.g. the allowed_failures matrix and env matrix?

@davetcoleman
Copy link
Member

@130s now that ros-industrial/industrial_ci#35 is merged should we update this to not include the duplicate pre-release functionality?

@130s
Copy link
Contributor Author

130s commented May 6, 2016

@davetcoleman ya, #292 is it ;)

@davetcoleman
Copy link
Member

oops haven't gotten through all my emails

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants