ROSS icon indicating copy to clipboard operation
ROSS copied to clipboard

Published 20 hours ago • verified publisher icon ROSS-org

travis updates

Open gonsie opened this issue 3 years ago • 1 comments

THIS PR SHOULD NOT BE MERGED

I’m update some of the travis configurations. I need the tests to run for master branch PRs, which is why I’m opening this PR. BUT, it is not a proper release and therefore should not be merged into master (as per our versioning and release guide

  1. Only deploy the doxygen when the master branch is updated. Currently, the doxygen is updated for PRs. Moving this section to deploy should mean that it is not run for PRs. But we may have to change the repo settings to also notify travis on master branch pushes.
  2. Build CODES for integration testing. The intent is to only do this on master branch PRs (aka releases).

gonsie avatar Oct 05 '20 23:10 gonsie

Finally got Travis to work properly, but found a bug with the CODES test suite (see CODES-org/CODES#206).

gonsie avatar Oct 06 '20 19:10 gonsie