Add a basic ci script.

Review Request #49 — Created March 4, 2014 and submitted

jsirois
commons
jsirois/pants/independent_ci_script
234
skumaran, travis, wickman
commit 983d2aa31f1448c971c5b902a0d9ba4cfd80fac3
Author: John Sirois <jsirois@twitter.com>
Date:   Tue Mar 4 10:05:26 2014 -0700

    Add a basic ci script.
    
    This allows for local emulation of ci whatever that happens to
    currently mean.

 .travis.yml             |  9 +------
 build-support/bin/ci.sh | 71 ++++++++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 72 insertions(+), 8 deletions(-)
$ ./build-support/bin/ci.sh

Also:
$ ./build-support/bin/ci.sh -h 
$ ./build-support/bin/ci.sh -bj # just python
$ ./build-support/bin/ci.sh -bdp # just java - no nailguns
WI
  1. Ship It!
  2. 
      
LA
  1. "local emulation of ci"
    
    I bet this is awesome. I dunno the intent, tho. I'm not sure when I, hypothetical open-source Pants developer, want to use it (versus tests/.../pants:all). Any guidance?
    
    (a) Just run this. It's so easy.
    
    (b) pants:all is usually fine. We only expect humans to run this ci.sh script in rare circumstances.
    
    (c) Other: ________________.
    1. the impetus for this change is that I was running "./pants py src/python/twitter/pants -- tests/python/twitter/pants:all" in order to test the big BUILD refactor, which apparently missed a few things.  it turns out I should've been executing the commands in .travis.yml.  so I think it's a combination of (a) and (c) other: any time you make a large structural change to pants.
    2. (c) before posting a review or merging a branch.  If you're not sure what you might have broke, run with no args, if you know what you can safely skip (jvm or python), use the skip flags as documented by -h.
    3. Cool beans, thank you.
  2. 
      
LA
  1. Ship It!
  2. 
      
SK
  1. Ship It!
  2. 
      
JS
  1. Thanks folks - submitted.
  2. 
      
JS
Review request changed

Status: Closed (submitted)

Loading...