[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.

Generated at Sat Feb 10 15:48:47 JST 2024 using Jira 8.3.4#803005-sha1:1f96e09b3c60279a408a2ae47be3c745f571388b.