----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20364/#review40365 -----------------------------------------------------------
Patch looks great! Reviews applied: [20364] All tests passed. - Mesos ReviewBot 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 > >
