threads / patch / 62368

v2t7011: ensure no whitespace after redirect

Subject: [PATCH v2 Outreachy] t7011: ensure no whitespace after redirect

## tl;dr

9 messages between Oct 19, 2024 and Oct 23, 2024. Diffs are folded; open one to read it.

replies: 8people: 4as markdown or json

Seyi Kuforiji· Oct 19, 2024, 16:34 UTC · lore

This change updates the script to conform to the coding standards outlined in the Git project's documentation. According to the guidelines in Documentation/CodingGuidelines under "Redirection operators", there should be no whitespace after redirection operators.

Signed-off-by: Seyi Kuforiji <kuforiji98@gmail.com>
---
 t/t7011-skip-worktree-reading.sh | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)
Show changes to t/t7011-skip-worktree-reading.sh +11 −11
diff --git a/t/t7011-skip-worktree-reading.sh b/t/t7011-skip-worktree-reading.sh
index 4adac5acd5..c86abd99bf 100755
--- a/t/t7011-skip-worktree-reading.sh
+++ b/t/t7011-skip-worktree-reading.sh
@@ -32,24 +32,24 @@ setup_absent() {
 }
 
 test_absent() {
-	echo "100644 $EMPTY_BLOB 0	1" > expected &&
-	git ls-files --stage 1 > result &&
+	echo "100644 $EMPTY_BLOB 0	1" >expected &&
+	git ls-files --stage 1 >result &&
 	test_cmp expected result &&
 	test ! -f 1
 }
 
 setup_dirty() {
 	git update-index --force-remove 1 &&
-	echo dirty > 1 &&
+	echo dirty >1 &&
 	git update-index --add --cacheinfo 100644 $EMPTY_BLOB 1 &&
 	git update-index --skip-worktree 1
 }
 
 test_dirty() {
-	echo "100644 $EMPTY_BLOB 0	1" > expected &&
-	git ls-files --stage 1 > result &&
+	echo "100644 $EMPTY_BLOB 0	1" >expected &&
+	git ls-files --stage 1 >result &&
 	test_cmp expected result &&
-	echo dirty > expected
+	echo dirty >expected
 	test_cmp expected 1
 }
 
@@ -59,7 +59,7 @@ test_expect_success 'setup' '
 	touch ./1 ./2 sub/1 sub/2 &&
 	git add 1 2 sub/1 sub/2 &&
 	git update-index --skip-worktree 1 sub/1 &&
-	git ls-files -t > result &&
+	git ls-files -t >result &&
 	test_cmp expect.skip result
 '
 
@@ -86,7 +86,7 @@ test_expect_success 'update-index --remove' '
 	setup_dirty &&
 	git update-index --remove 1 &&
 	test -z "$(git ls-files 1)" &&
-	echo dirty > expected &&
+	echo dirty >expected &&
 	test_cmp expected 1
 '
 
@@ -110,16 +110,16 @@ test_expect_success 'ls-files --modified' '
 	test -z "$(git ls-files -m)"
 '
 
-echo ":000000 100644 $ZERO_OID $EMPTY_BLOB A	1" > expected
+echo ":000000 100644 $ZERO_OID $EMPTY_BLOB A	1" >expected
 test_expect_success 'diff-index does not examine skip-worktree absent entries' '
 	setup_absent &&
-	git diff-index HEAD -- 1 > result &&
+	git diff-index HEAD -- 1 >result &&
 	test_cmp expected result
 '
 
 test_expect_success 'diff-index does not examine skip-worktree dirty entries' '
 	setup_dirty &&
-	git diff-index HEAD -- 1 > result &&
+	git diff-index HEAD -- 1 >result &&
 	test_cmp expected result
 '
 

-- 
2.47.0.86.g15030f9556
Kristoffer Haugsbakk· Oct 19, 2024, 17:54 UTC · re: Seyi Kuforiji · lore

Re: [PATCH v2 Outreachy] t7011: ensure no whitespace after redirect

On Sat, Oct 19, 2024, at 18:34, Seyi Kuforiji wrote:
Show 9 quoted lines
> This change updates the script to conform to the coding standards
> outlined in the Git project's documentation. According to the guidelines
> in Documentation/CodingGuidelines under "Redirection operators", there
> should be no whitespace after redirection operators.
>
> Signed-off-by: Seyi Kuforiji <kuforiji98@gmail.com>
> ---
>  t/t7011-skip-worktree-reading.sh | 22 +++++++++++-----------
>  1 file changed, 11 insertions(+), 11 deletions(-)

I can confirm that there are no changes in this version to the patch itself. Which was expected since the change was good.

The difference in this version is that the commit message has been improved. At least according to my taste.

Thanks
-- 
Kristoffer Haugsbakk
Patrick Steinhardt· Oct 21, 2024, 12:03 UTC · re: Kristoffer Haugsbakk · lore

Re: [PATCH v2 Outreachy] t7011: ensure no whitespace after redirect

On Sat, Oct 19, 2024 at 07:54:30PM +0200, Kristoffer Haugsbakk wrote:
Show 16 quoted lines
> On Sat, Oct 19, 2024, at 18:34, Seyi Kuforiji wrote:
> > This change updates the script to conform to the coding standards
> > outlined in the Git project's documentation. According to the guidelines
> > in Documentation/CodingGuidelines under "Redirection operators", there
> > should be no whitespace after redirection operators.
> >
> > Signed-off-by: Seyi Kuforiji <kuforiji98@gmail.com>
> > ---
> >  t/t7011-skip-worktree-reading.sh | 22 +++++++++++-----------
> >  1 file changed, 11 insertions(+), 11 deletions(-)
> 
> I can confirm that there are no changes in this version to the patch
> itself.  Which was expected since the change was good.
> 
> The difference in this version is that the commit message has been
> improved.  At least according to my taste.
Agreed, this patch looks good to me, too. Thanks!
Patrick
Seyi Chamber· Oct 21, 2024, 17:17 UTC · re: Patrick Steinhardt · lore

Re: [PATCH v2 Outreachy] t7011: ensure no whitespace after redirect

On Mon, 21 Oct 2024 at 13:03, Patrick Steinhardt <ps@pks.im> wrote:
Show 22 quoted lines
>
> On Sat, Oct 19, 2024 at 07:54:30PM +0200, Kristoffer Haugsbakk wrote:
> > On Sat, Oct 19, 2024, at 18:34, Seyi Kuforiji wrote:
> > > This change updates the script to conform to the coding standards
> > > outlined in the Git project's documentation. According to the guidelines
> > > in Documentation/CodingGuidelines under "Redirection operators", there
> > > should be no whitespace after redirection operators.
> > >
> > > Signed-off-by: Seyi Kuforiji <kuforiji98@gmail.com>
> > > ---
> > >  t/t7011-skip-worktree-reading.sh | 22 +++++++++++-----------
> > >  1 file changed, 11 insertions(+), 11 deletions(-)
> >
> > I can confirm that there are no changes in this version to the patch
> > itself.  Which was expected since the change was good.
> >
> > The difference in this version is that the commit message has been
> > improved.  At least according to my taste.
>
> Agreed, this patch looks good to me, too. Thanks!
>
> Patrick
Yay!
Thank you for the feedback @Patrick Steinhardt @Kristoffer Haugsbakk
Regards
Seyi
Taylor Blau· Oct 21, 2024, 19:58 UTC · re: Seyi Chamber · lore

Re: [PATCH v2 Outreachy] t7011: ensure no whitespace after redirect

On Mon, Oct 21, 2024 at 06:17:06PM +0100, Seyi Chamber wrote:
> Yay!
>
> Thank you for the feedback @Patrick Steinhardt @Kristoffer Haugsbakk
Thanks, all. Let's start merging this one down.

Thanks, Taylor

Seyi Chamber· Oct 22, 2024, 17:11 UTC · re: Taylor Blau · lore

Re: [PATCH v2 Outreachy] t7011: ensure no whitespace after redirect

On Mon, 21 Oct 2024 at 20:58, Taylor Blau <me@ttaylorr.com> wrote:
Show 10 quoted lines
>
> On Mon, Oct 21, 2024 at 06:17:06PM +0100, Seyi Chamber wrote:
> > Yay!
> >
> > Thank you for the feedback @Patrick Steinhardt @Kristoffer Haugsbakk
>
> Thanks, all. Let's start merging this one down.
>
> Thanks,
> Taylor
Thanks, Taylor!

I've also discovered that the test script t9101-git-svn-props.sh also requires the same update done in this patch. Is this test script currently being worked on? @Patrick Steinhardt

Thanks Seyi

Patrick Steinhardt· Oct 23, 2024, 05:00 UTC · re: Seyi Chamber · lore

Re: [PATCH v2 Outreachy] t7011: ensure no whitespace after redirect

On Tue, Oct 22, 2024 at 06:11:17PM +0100, Seyi Chamber wrote:
Show 17 quoted lines
> On Mon, 21 Oct 2024 at 20:58, Taylor Blau <me@ttaylorr.com> wrote:
> >
> > On Mon, Oct 21, 2024 at 06:17:06PM +0100, Seyi Chamber wrote:
> > > Yay!
> > >
> > > Thank you for the feedback @Patrick Steinhardt @Kristoffer Haugsbakk
> >
> > Thanks, all. Let's start merging this one down.
> >
> > Thanks,
> > Taylor
> 
> Thanks, Taylor!
> 
> I've also discovered that the test script t9101-git-svn-props.sh also
> requires the same update done in this patch.
> Is this test script currently being worked on? @Patrick Steinhardt

Not that I'd know of. I typically double check via lore by searching for the relevant subsystem [1]. This only surfaces your mail and the Meson patch series, so at least nobody else has communicated intent to work on it.

Patrick
[1]: https://lore.kernel.org/git/?q=t9101
Seyi Chamber· Oct 23, 2024, 07:24 UTC · re: Patrick Steinhardt · lore

Re: [PATCH v2 Outreachy] t7011: ensure no whitespace after redirect

On Wed, 23 Oct 2024 at 06:00, Patrick Steinhardt <ps@pks.im> wrote:
Show 28 quoted lines
>
> On Tue, Oct 22, 2024 at 06:11:17PM +0100, Seyi Chamber wrote:
> > On Mon, 21 Oct 2024 at 20:58, Taylor Blau <me@ttaylorr.com> wrote:
> > >
> > > On Mon, Oct 21, 2024 at 06:17:06PM +0100, Seyi Chamber wrote:
> > > > Yay!
> > > >
> > > > Thank you for the feedback @Patrick Steinhardt @Kristoffer Haugsbakk
> > >
> > > Thanks, all. Let's start merging this one down.
> > >
> > > Thanks,
> > > Taylor
> >
> > Thanks, Taylor!
> >
> > I've also discovered that the test script t9101-git-svn-props.sh also
> > requires the same update done in this patch.
> > Is this test script currently being worked on? @Patrick Steinhardt
>
> Not that I'd know of. I typically double check via lore by searching for
> the relevant subsystem [1]. This only surfaces your mail and the Meson
> patch series, so at least nobody else has communicated intent to work on
> it.
>
> Patrick
>
> [1]: https://lore.kernel.org/git/?q=t9101
Thanks, Patrick.

In the future, would you prefer I still run it by you before working on a test script, or should I apply the same strategy you mentioned in your reply?

Regards
Seyi
Patrick Steinhardt· Oct 23, 2024, 09:55 UTC · re: Seyi Chamber · lore

Re: [PATCH v2 Outreachy] t7011: ensure no whitespace after redirect

On Wed, Oct 23, 2024 at 08:24:18AM +0100, Seyi Chamber wrote:
Show 35 quoted lines
> On Wed, 23 Oct 2024 at 06:00, Patrick Steinhardt <ps@pks.im> wrote:
> >
> > On Tue, Oct 22, 2024 at 06:11:17PM +0100, Seyi Chamber wrote:
> > > On Mon, 21 Oct 2024 at 20:58, Taylor Blau <me@ttaylorr.com> wrote:
> > > >
> > > > On Mon, Oct 21, 2024 at 06:17:06PM +0100, Seyi Chamber wrote:
> > > > > Yay!
> > > > >
> > > > > Thank you for the feedback @Patrick Steinhardt @Kristoffer Haugsbakk
> > > >
> > > > Thanks, all. Let's start merging this one down.
> > > >
> > > > Thanks,
> > > > Taylor
> > >
> > > Thanks, Taylor!
> > >
> > > I've also discovered that the test script t9101-git-svn-props.sh also
> > > requires the same update done in this patch.
> > > Is this test script currently being worked on? @Patrick Steinhardt
> >
> > Not that I'd know of. I typically double check via lore by searching for
> > the relevant subsystem [1]. This only surfaces your mail and the Meson
> > patch series, so at least nobody else has communicated intent to work on
> > it.
> >
> > Patrick
> >
> > [1]: https://lore.kernel.org/git/?q=t9101
> 
> Thanks, Patrick.
> 
> In the future, would you prefer I still run it by you before working
> on a test script,
> or should I apply the same strategy you mentioned in your reply?

There's no need to run it by anybody, so please feel free to use the same strategy.

Patrick

← back to recent threads