[PIPE2D-134] Enable Travis on push to master only Created: 01/Dec/16 Updated: 02/Dec/16 Resolved: 02/Dec/16 |
|
| Status: | Done |
| Project: | DRP 2-D Pipeline |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Story | Priority: | Major |
| Reporter: | price | Assignee: | price |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Reviewers: | swinbank |
| Description |
|
I currently have Travis set up to build only pull requests. This leaves the little build status indicator on Travis as "unknown". I would like to have Travis fire on pull requests and pushes to master, like this. |
| Comments |
| Comment by price [ 01/Dec/16 ] |
|
swinbank, would you please have a look at this minor fix to Travis:
price@price-laptop:~/PFS/pfs_pipe2d (tickets/PIPE2D-134=) $ git sub-patch
commit 36b420585fe63b91dd8546b3b277b8d77b602416
Author: Paul Price <price@astro.princeton.edu>
Date: Wed Nov 30 22:27:43 2016 -0500
Travis: build on pushes to master branch
Currently, we're building only pull requests.
This also requires activating push builds on Travis.
diff --git a/.travis.yml b/.travis.yml
index 5d1eca5..6b742c9 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -14,6 +14,9 @@ matrix:
fast_finish: true
notifications:
email: false
+branches:
+ only:
+ - master
# The actual installation is done under the 'script' so as to consolidate the
# necessary front-end machinery (intercept log, decide which branch to use).
diff --git a/travis.yml.TEMPLATE b/travis.yml.TEMPLATE
index 657e0e0..5a207dc 100644
--- a/travis.yml.TEMPLATE
+++ b/travis.yml.TEMPLATE
@@ -12,6 +12,9 @@ matrix:
fast_finish: true
notifications:
email: false
+branches:
+ only:
+ - master
# The actual installation is done under the 'script' so as to consolidate the
# necessary front-end machinery (intercept log, decide which branch to use).
(The other packages have equivalent changes.) |
| Comment by price [ 01/Dec/16 ] |
|
(I still need to enable push builds in Travis. Saving that until after this lands on master so we don't generate more builds than necessary.) |
| Comment by swinbank [ 01/Dec/16 ] |
|
Looks fine. |
| Comment by price [ 02/Dec/16 ] |
|
Thanks John. Enabled push builds in Travis, merged to master and now everything's building. |