[PATCH] SubmittingPatches: Document how to request a patch review tag

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Document the preferred way a developer should request to have their
Acked-by/Tested-by/Reviewed-by tag to a patch series under discussion

Signed-off-by: Jason Holden <jason.k.holden.swdev@xxxxxxxxx>
---
Junio,
  I was ready to add my Reviewed-by to this patch series, but I wasn't sure if
I should email just you the patch author (to cut down on overall list traffic)
or both you and the list.  If all reviewed-by/acked-by/tested-by traffic 
should go via the email list I think this patch would be helpful, as I 
wasn't quite sure how wide of a distribution list to use for my 
"Reviewed-by" email.

A very similiar question was asked previously in:
http://thread.gmane.org/gmane.comp.version-control.git/185564/focus=185570

This will apply on top of your last tweak to SubmittingPatches

Please add my reviewed-by to the rest of the patches in this series.

-Jason

 Documentation/SubmittingPatches | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/Documentation/SubmittingPatches b/Documentation/SubmittingPatches
index f6276ff..80001c9 100644
--- a/Documentation/SubmittingPatches
+++ b/Documentation/SubmittingPatches
@@ -268,6 +268,11 @@ If you like, you can put extra tags at the end:
 4. "Tested-by:" is used to indicate that the person applied the patch
    and found it to have the desired effect.
 
+If you are a reviewer and wish to add your Acked-by/Reviewed-by/Tested-by tag
+to a patch series under discussion (after having reviewed it or tested it
+of course!), reply to the author of the patch series, cc'ing the git mailing
+list.
+
 You can also create your own tag or use one that's in common usage
 such as "Thanks-to:", "Based-on-patch-by:", or "Mentored-by:".
 
-- 
1.8.1.rc3.28.g0ab5d1f

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]