From: Junio C Hamano <gitster@pobox.com>
To: AbdAlRahman Gad <abdobngad@gmail.com>
Cc: git@vger.kernel.org
Subject: Re: [PATCH v4 0/8] t7004: Modernize the style
Date: Mon, 05 Aug 2024 17:45:15 -0700 [thread overview]
Message-ID: <xmqqzfpqzdgk.fsf@gitster.g> (raw)
In-Reply-To: <20240805235917.190699-1-abdobngad@gmail.com> (AbdAlRahman Gad's message of "Tue, 6 Aug 2024 02:59:09 +0300")
AbdAlRahman Gad <abdobngad@gmail.com> writes:
> AbdAlRahman Gad (8):
> [Newcomer] t7004: Remove space after redirect operators
> [Newcomer] t7004: Do not lose exit status to pipe
> [Newcomer] t7004: One command per line
> [Newcomer] t7004: Do not prepare things outside test_expect_success
> [Newcomer] t7004: Use indented here-doc
> [Newcomer] t7004: Description on the same line as test_expect_success
> [Newcomer] t7004: Description and test body seperated with backslash
> [Newcomer] t7004: Use single quotes instead of double quotes
As you can see from "git shortlog --no-merges --since=6.months", we
do not capitalize the first word of the title description of each
patch.
Also, I haven't looked at the individual patches, it seems the patch
is somewhoat whitespace damaged.
$ git am -s ./+ag8-v4
Applying: t7004: remove space after redirect operators
.git/rebase-apply/patch:169: indent with spaces.
git checkout stable &&
.git/rebase-apply/patch:193: indent with spaces.
git tag v4.0
warning: 2 lines add whitespace errors.
Applying: t7004: do not lose exit status to pipe
Applying: t7004: one command per line
Applying: t7004: do not prepare things outside test_expect_success
.git/rebase-apply/patch:1188: indent with spaces.
git tag v4.0
warning: 1 line adds whitespace errors.
Applying: t7004: use indented here-doc
Applying: t7004: description on the same line as test_expect_success
Applying: t7004: test description and test body seperated with backslash
Applying: t7004: use single quotes instead of double quotes
.git/rebase-apply/patch:133: indent with spaces.
git tag v4.0
warning: 1 line adds whitespace errors.
next prev parent reply other threads:[~2024-08-06 0:45 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-08-05 23:59 [PATCH v4 0/8] t7004: Modernize the style AbdAlRahman Gad
2024-08-05 23:59 ` [PATCH v4 1/8] [Newcomer] t7004: Remove space after redirect operators AbdAlRahman Gad
2024-08-05 23:59 ` [PATCH v4 2/8] [Newcomer] t7004: Do not lose exit status to pipe AbdAlRahman Gad
2024-08-06 3:13 ` Eric Sunshine
2024-08-06 8:38 ` AbdAlRahman Gad
2024-08-06 18:05 ` Eric Sunshine
2024-08-05 23:59 ` [PATCH v4 3/8] [Newcomer] t7004: One command per line AbdAlRahman Gad
2024-08-05 23:59 ` [PATCH v4 4/8] [Newcomer] t7004: Do not prepare things outside test_expect_success AbdAlRahman Gad
2024-08-06 3:29 ` Eric Sunshine
2024-08-06 8:41 ` AbdAlRahman Gad
2024-08-05 23:59 ` [PATCH v4 5/8] [Newcomer] t7004: Use indented here-doc AbdAlRahman Gad
2024-08-05 23:59 ` [PATCH v4 6/8] [Newcomer] t7004: Description on the same line as test_expect_success AbdAlRahman Gad
2024-08-05 23:59 ` [PATCH v4 7/8] [Newcomer] t7004: test Description and test body seperated with backslash AbdAlRahman Gad
2024-08-05 23:59 ` [PATCH v4 8/8] [Newcomer] t7004: Use single quotes instead of double quotes AbdAlRahman Gad
2024-08-06 3:02 ` Eric Sunshine
2024-08-06 10:10 ` AbdAlRahman Gad
2024-08-06 17:21 ` Eric Sunshine
2024-08-06 16:35 ` Junio C Hamano
2024-08-06 17:11 ` Eric Sunshine
2024-08-06 0:45 ` Junio C Hamano [this message]
2024-08-06 8:44 ` [PATCH v4 0/8] t7004: Modernize the style AbdAlRahman Gad
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=xmqqzfpqzdgk.fsf@gitster.g \
--to=gitster@pobox.com \
--cc=abdobngad@gmail.com \
--cc=git@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.