Skip to content

Commit 030f066

Browse files
krzkJonathan Corbet
authored andcommitted
docs: submitting-patches: describe preserving review/test tags
From time to time, the novice kernel contributors do not add Reviewed-by or Tested-by tags to the next versions of the patches. Mostly because they are unaware that responsibility of adding these tags in next version is on submitter, not maintainer. Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org> Link: https://lore.kernel.org/r/20201013162725.13572-1-krzk@kernel.org Signed-off-by: Jonathan Corbet <corbet@lwn.net>
1 parent e0533de commit 030f066

1 file changed

Lines changed: 7 additions & 0 deletions

File tree

Documentation/process/submitting-patches.rst

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -527,6 +527,13 @@ done on the patch. Reviewed-by: tags, when supplied by reviewers known to
527527
understand the subject area and to perform thorough reviews, will normally
528528
increase the likelihood of your patch getting into the kernel.
529529

530+
Both Tested-by and Reviewed-by tags, once received on mailing list from tester
531+
or reviewer, should be added by author to the applicable patches when sending
532+
next versions. However if the patch has changed substantially in following
533+
version, these tags might not be applicable anymore and thus should be removed.
534+
Usually removal of someone's Tested-by or Reviewed-by tags should be mentioned
535+
in the patch changelog (after the '---' separator).
536+
530537
A Suggested-by: tag indicates that the patch idea is suggested by the person
531538
named and ensures credit to the person for the idea. Please note that this
532539
tag should not be added without the reporter's permission, especially if the

0 commit comments

Comments
 (0)