# [PATCH v2 Outreachy] t7011: ensure no whitespace after redirect 9 messages from 2024-10-19 to 2024-10-23. Participants: Seyi Kuforiji, Kristoffer Haugsbakk, Patrick Steinhardt, Seyi Chamber, Taylor Blau. Thread: https://gitlist.dev/t/62368 ## Seyi Kuforiji, 2024-10-19 16:34 Subject: [PATCH v2 Outreachy] t7011: ensure no whitespace after redirect Message-ID: <20241019163439.274656-1-kuforiji98@gmail.com> URL: https://gitlist.dev/e/20241019163439.274656-1-kuforiji98%40gmail.com ``` 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 --- t/t7011-skip-worktree-reading.sh | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) 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, 2024-10-19 17:54 Subject: Re: [PATCH v2 Outreachy] t7011: ensure no whitespace after redirect Message-ID: URL: https://gitlist.dev/e/cb88194a-15d2-4c3d-8768-189853572ac0%40app.fastmail.com In-Reply-To: <20241019163439.274656-1-kuforiji98@gmail.com> ``` 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 > --- > 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, 2024-10-21 12:03 Subject: Re: [PATCH v2 Outreachy] t7011: ensure no whitespace after redirect Message-ID: URL: https://gitlist.dev/e/ZxZDFyKxTn_D7c3o%40pks.im In-Reply-To: ``` 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 > > --- > > 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, 2024-10-21 17:17 Subject: Re: [PATCH v2 Outreachy] t7011: ensure no whitespace after redirect Message-ID: URL: https://gitlist.dev/e/CAGedMteMtaOvyE2gX59bMudEUmCZw7nOh%3DMuDRi%3D7CbYm-gBDQ%40mail.gmail.com In-Reply-To: ``` On Mon, 21 Oct 2024 at 13:03, Patrick Steinhardt wrote: > > 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 > > > --- > > > 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, 2024-10-21 19:58 Subject: Re: [PATCH v2 Outreachy] t7011: ensure no whitespace after redirect Message-ID: URL: https://gitlist.dev/e/ZxaySpwA7jg7c1aw%40nand.local In-Reply-To: ``` 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, 2024-10-22 17:11 Subject: Re: [PATCH v2 Outreachy] t7011: ensure no whitespace after redirect Message-ID: URL: https://gitlist.dev/e/CAGedMtea3_65wuOc1c548TC1x%3DThirB1gC8p6kTomwWf_VzDng%40mail.gmail.com In-Reply-To: ``` On Mon, 21 Oct 2024 at 20:58, Taylor Blau 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 Thanks Seyi ``` ## Patrick Steinhardt, 2024-10-23 05:00 Subject: Re: [PATCH v2 Outreachy] t7011: ensure no whitespace after redirect Message-ID: URL: https://gitlist.dev/e/ZxiC06eY4mMh1Xun%40pks.im In-Reply-To: ``` On Tue, Oct 22, 2024 at 06:11:17PM +0100, Seyi Chamber wrote: > On Mon, 21 Oct 2024 at 20:58, Taylor Blau 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, 2024-10-23 07:24 Subject: Re: [PATCH v2 Outreachy] t7011: ensure no whitespace after redirect Message-ID: URL: https://gitlist.dev/e/CAGedMtdN8qRRALiYdCFso90ExGEm0zirnpLXuva_bc8ehsK%2BXw%40mail.gmail.com In-Reply-To: ``` On Wed, 23 Oct 2024 at 06:00, Patrick Steinhardt wrote: > > On Tue, Oct 22, 2024 at 06:11:17PM +0100, Seyi Chamber wrote: > > On Mon, 21 Oct 2024 at 20:58, Taylor Blau 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, 2024-10-23 09:55 Subject: Re: [PATCH v2 Outreachy] t7011: ensure no whitespace after redirect Message-ID: URL: https://gitlist.dev/e/ZxjIC5W4HprJqUxB%40pks.im In-Reply-To: ``` On Wed, Oct 23, 2024 at 08:24:18AM +0100, Seyi Chamber wrote: > On Wed, 23 Oct 2024 at 06:00, Patrick Steinhardt wrote: > > > > On Tue, Oct 22, 2024 at 06:11:17PM +0100, Seyi Chamber wrote: > > > On Mon, 21 Oct 2024 at 20:58, Taylor Blau 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 ```