mirror of
https://github.com/ceph/ceph
synced 2025-02-21 01:47:25 +00:00
Merge pull request #4882 from SUSE/wip-submitting-patches-clarification
SubmittingPatches: clarify how Reviewed-by lines are added Reviewed-by: Sage Weil <sage@redhat.com>
This commit is contained in:
commit
efededa010
@ -273,6 +273,15 @@ A: The target branch depends on the nature of your change:
|
|||||||
you are fixing a bug that exists on the "firefly" branch and that you
|
you are fixing a bug that exists on the "firefly" branch and that you
|
||||||
desire that your change be cherry-picked to that branch.
|
desire that your change be cherry-picked to that branch.
|
||||||
|
|
||||||
|
Q: How to include Reviewed-by: tag(s) in my pull request?
|
||||||
|
|
||||||
|
You don't. If someone reviews your pull request, they should indicate they
|
||||||
|
have done so by commenting on it with "+1", "looks good to me", "LGTM",
|
||||||
|
and/or the entire "Reviewed-by: ..." line with their name and email address.
|
||||||
|
|
||||||
|
The developer merging the pull request should note positive reviews and
|
||||||
|
include the appropriate Reviewed-by: lines in the merge commit.
|
||||||
|
|
||||||
2) Patch submission via ceph-devel@vger.kernel.org
|
2) Patch submission via ceph-devel@vger.kernel.org
|
||||||
|
|
||||||
The best way to generate a patch for manual submission is to work from
|
The best way to generate a patch for manual submission is to work from
|
||||||
|
Loading…
Reference in New Issue
Block a user