----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43552/#review120223 -----------------------------------------------------------
support/push-reviews.py (lines 14 - 15) <https://reviews.apache.org/r/43552/#comment181647> I would at least pass this as a flag right now rather than jsut adding a TODO. Otherwise people that prefer to close with the commit message won't be able to use your script at all. support/push-reviews.py (lines 123 - 125) <https://reviews.apache.org/r/43552/#comment181648> Shouldn't this also be protected by the `if not options['dry_run']` check? - Kevin Klues On Feb. 22, 2016, 4:52 a.m., Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/43552/ > ----------------------------------------------------------- > > (Updated Feb. 22, 2016, 4:52 a.m.) > > > Review request for mesos, Artem Harutyunyan, Kevin Klues, and Michael Park. > > > Bugs: MESOS-3929 > https://issues.apache.org/jira/browse/MESOS-3929 > > > Repository: mesos > > > Description > ------- > > This script allows committers to push locally applied review chain to the ASF > git repo and mark the reviews as submitted. > > > Diffs > ----- > > support/push-reviews.py PRE-CREATION > > Diff: https://reviews.apache.org/r/43552/diff/ > > > Testing > ------- > > Tested locally. > > > Thanks, > > Vinod Kone > >
