From: Eric Sunshine <sunshine@sunshineco.com>
Date: Mon, 18 Mar 2019 20:32:06 GMT
Subject: Re: [PATCH v8 10/11] sequencer.c: define describe_cleanup_mode
Message-ID: <CAPig+cSAVLh7eahoqitC0tdYdDa6j4=iBZMiNw3ab-ysR=i_Cw@mail.gmail.com>
In-Reply-To: <20190318203007.GA14875@dev-l>

On Mon, Mar 18, 2019 at 4:30 PM Denton Liu <liu.denton@gmail.com> wrote:
> On Mon, Mar 18, 2019 at 04:04:22PM -0400, Eric Sunshine wrote:
> > These two Reviewed-by: lines should be dropped for a couple reasons.
> >
> I was using it to give credit for changes that I incorporated in
> response to a review. In addition, it was helpful for bookkeeping who
> suggested what where.
>
> Would a Helped-by tag be more appropriate in this situation?

Using Helped-by: would be quite appropriate.