Skip to content

Commit c752e7f

Browse files
committed
Merge branch 'maint'
* maint: t/t3415: use && where applicable. SubmittingPatches: Document some extra tags used in commit messages
2 parents 8520913 + 7c6eafa commit c752e7f

File tree

2 files changed

+18
-9
lines changed

2 files changed

+18
-9
lines changed

Documentation/SubmittingPatches

Lines changed: 15 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -264,12 +264,21 @@ the change to its true author (see (2) above).
264264
Also notice that a real name is used in the Signed-off-by: line. Please
265265
don't hide your real name.
266266

267-
Some people also put extra tags at the end.
268-
269-
"Acked-by:" says that the patch was reviewed by the person who
270-
is more familiar with the issues and the area the patch attempts
271-
to modify. "Tested-by:" says the patch was tested by the person
272-
and found to have the desired effect.
267+
If you like, you can put extra tags at the end:
268+
269+
1. "Reported-by:" is used to to credit someone who found the bug that
270+
the patch attempts to fix.
271+
2. "Acked-by:" says that the person who is more familiar with the area
272+
the patch attempts to modify liked the patch.
273+
3. "Reviewed-by:", unlike the other tags, can only be offered by the
274+
reviewer and means that she is completely satisfied that the patch
275+
is ready for application. It is usually offered only after a
276+
detailed review.
277+
4. "Tested-by:" is used to indicate that the person applied the patch
278+
and found it to have the desired effect.
279+
280+
You can also create your own tag or use one that's in common usage
281+
such as "Thanks-to:", "Based-on-patch-by:", or "Mentored-by:".
273282

274283
------------------------------------------------
275284
An ideal patch flow

t/t3415-rebase-autosquash.sh

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ test_auto_fixup() {
2626
echo 1 >file1 &&
2727
git add -u &&
2828
test_tick &&
29-
git commit -m "fixup! first"
29+
git commit -m "fixup! first" &&
3030

3131
git tag $1 &&
3232
test_tick &&
@@ -55,7 +55,7 @@ test_auto_squash() {
5555
echo 1 >file1 &&
5656
git add -u &&
5757
test_tick &&
58-
git commit -m "squash! first"
58+
git commit -m "squash! first" &&
5959

6060
git tag $1 &&
6161
test_tick &&
@@ -84,7 +84,7 @@ test_expect_success 'misspelled auto squash' '
8484
echo 1 >file1 &&
8585
git add -u &&
8686
test_tick &&
87-
git commit -m "squash! forst"
87+
git commit -m "squash! forst" &&
8888
git tag final-missquash &&
8989
test_tick &&
9090
git rebase --autosquash -i HEAD^^^ &&

0 commit comments

Comments
 (0)