On Thu, 2017-03-30 at 01:48 -0700, Nicholas A. Bellinger wrote: > Just for future reference, the flow of these tags should reflect the > history of the patch. Eg: > > Reviewed-by: First reviewer <[email protected]> > Tested-by: First tester <[email protected]> > Reviewed-by: Second reviewer <[email protected]> > Signed-off-by: Patch Author <[email protected]> > > and then once the subsystem maintainer merges it into his tree, they add > their own: > > Signed-off-by: Subsystem Maintainer <[email protected]>
Hi Nic, I agree that these tags should reflect the history of the patch. I think that means that the patch author should be mentioned first, Reviewed-by / Tested-by tags next and the subsystem maintainer sign-off last. At least, that's how most other maintainers do it. Bart.
