Re: Removing `support/apply-review.sh`

2017-01-13 Thread Joseph Wu
+1 for one less character to type while tab-completing:
support/ap  s

On Fri, Jan 13, 2017 at 10:15 AM, Vinod Kone  wrote:

> +1 to remove
>
> On Fri, Jan 13, 2017 at 1:39 AM, haosdent  wrote:
>
> > +1 for remove this.
> >
> > On Fri, Jan 13, 2017 at 5:37 PM, Michael Park  wrote:
> >
> > > Does anyone still care about `support/apply-review.sh`?
> > > I imagine most people have transitioned to `support/apply-reviews.py`.
> > >
> > > Please let me know if people still want it around for some reason.
> > >
> > > Thanks,
> > >
> > > MPark
> > >
> >
> >
> >
> > --
> > Best Regards,
> > Haosdent Huang
> >
>


Re: Removing `support/apply-review.sh`

2017-01-13 Thread Vinod Kone
+1 to remove

On Fri, Jan 13, 2017 at 1:39 AM, haosdent  wrote:

> +1 for remove this.
>
> On Fri, Jan 13, 2017 at 5:37 PM, Michael Park  wrote:
>
> > Does anyone still care about `support/apply-review.sh`?
> > I imagine most people have transitioned to `support/apply-reviews.py`.
> >
> > Please let me know if people still want it around for some reason.
> >
> > Thanks,
> >
> > MPark
> >
>
>
>
> --
> Best Regards,
> Haosdent Huang
>


Re: Removing `support/apply-review.sh`

2017-01-13 Thread haosdent
+1 for remove this.

On Fri, Jan 13, 2017 at 5:37 PM, Michael Park  wrote:

> Does anyone still care about `support/apply-review.sh`?
> I imagine most people have transitioned to `support/apply-reviews.py`.
>
> Please let me know if people still want it around for some reason.
>
> Thanks,
>
> MPark
>



-- 
Best Regards,
Haosdent Huang


Removing `support/apply-review.sh`

2017-01-13 Thread Michael Park
Does anyone still care about `support/apply-review.sh`?
I imagine most people have transitioned to `support/apply-reviews.py`.

Please let me know if people still want it around for some reason.

Thanks,

MPark