Thread (99 messages) flat view 99 messages, 7 authors, 2016-12-15

Re: [PATCH v3] transport: add protocol policy config option

From: Jeff King <hidden>
Date: 2016-11-04 23:09:09

On Fri, Nov 04, 2016 at 02:35:57PM -0700, Stefan Beller wrote:
On Fri, Nov 4, 2016 at 1:58 PM, Brandon Williams [off-list ref] wrote:
quoted
On 11/04, Brandon Williams wrote:
quoted
Signed-off-by: Brandon Williams <redacted>
Is there an acceptable way to give credit to Jeff for helping with this patch?
What about:
Helped-by: Jeff King [off-list ref]
That, or often I would write:

  Based on a patch by Jeff King [off-list ref]

in the commit message. Basically anything is OK _except_ forging
signed-off-by, because it has a very specific meaning. So let me also
say that I am happy to give my:

  Signed-off-by: Jeff King [off-list ref]

to the original (which you should add in, to make clear that the
copyright issues are OK).

In some cases it makes sense to just roll somebody's patch into your
series, and then build on top. I'm fine with it all going into a single
patch here.

-Peff
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help