----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20364/#review40401 -----------------------------------------------------------
Ship it! support/post-reviews.py <https://reviews.apache.org/r/20364/#comment73378> I think this is fine, but what about also checking if you're on the master branch and giving a nice error message that says we're expecting you to be working on another branch from master? Thanks for making the tools easier to use Chengwei! - Benjamin Hindman On April 15, 2014, 8:06 a.m., Chengwei Yang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/20364/ > ----------------------------------------------------------- > > (Updated April 15, 2014, 8:06 a.m.) > > > Review request for mesos, Adam B, Ben Mahler, Jie Yu, and Vinod Kone. > > > Repository: mesos-git > > > Description > ------- > > post-reviews.py: handle working on master branch > > This is a small fix but wanted most by me, whenever working on master > branch directly with small patches, rather than a dedicate branch, > post-reviews.py doesn't handle this situation but just fail with the > trace back like: > > % ./support/post-reviews.py > Running post-review across all of ... > Traceback (most recent call last): > File "./support/post-reviews.py", line 104, in <module> > sha = line.split()[0] > IndexError: list index out of range > > This is not quite user friendly or make a lot of sense. This patch will > handle the above situation and notice user useful information. > > > Diffs > ----- > > support/post-reviews.py 602e6600e65d2b122a2314694f6969a27b840c5d > > Diff: https://reviews.apache.org/r/20364/diff/ > > > Testing > ------- > > done. > > > Thanks, > > Chengwei Yang > >
