{"thread":{"id":"59285","subject":"[GSOC] [PATCH v1 0/2] Modernize t9700 test script","startedAt":"2023-02-22T04:08:11Z","lastAt":"2023-02-25T01:33:23Z","messageCount":14,"participants":["Zhang Yi","Vivan Garg","Junio C Hamano","Philip Oakley"],"isPatch":true,"patchVersion":1,"patchTotal":2},"messages":[{"id":"472427","messageId":"20230222040745.1511205-1-18994118902@163.com","threadId":"59285","inReplyTo":null,"subject":"[GSOC] [PATCH v1 0/2] Modernize t9700 test script","fromName":"Zhang Yi","fromEmail":"18994118902@163.com","sentAt":"2023-02-22T04:07:43Z","receivedAt":"2023-02-22T04:08:11Z","isPatch":true,"sender":{"key":"18994118902@163.com","avatar":"https://avatars.githubusercontent.com/u/48300302?v=4"},"body":"A title in the test script is in old style an a body is indented by spaces.I fix the title style and change indents to TAB.\n\nZhang Yi (2):\n  Fix title style\n  Change indents  to TAB\n\n t/t9700-perl-git.sh | 61 ++++++++++++++++++++++-----------------------\n 1 file changed, 30 insertions(+), 31 deletions(-)\n\n-- \n2.29.0-rc0\n\n"},{"id":"472428","messageId":"20230222040745.1511205-2-18994118902@163.com","threadId":"59285","inReplyTo":"20230222040745.1511205-1-18994118902@163.com","subject":"[GSOC] [PATCH v1 1/2] Fix title style","fromName":"Zhang Yi","fromEmail":"18994118902@163.com","sentAt":"2023-02-22T04:07:44Z","receivedAt":"2023-02-22T04:08:28Z","isPatch":true,"sender":{"key":"18994118902@163.com","avatar":"https://avatars.githubusercontent.com/u/48300302?v=4"},"body":"Signed-off-by: Zhang Yi <18994118902@163.com>\n---\n t/t9700-perl-git.sh | 5 ++---\n 1 file changed, 2 insertions(+), 3 deletions(-)\n\ndiff --git a/t/t9700-perl-git.sh b/t/t9700-perl-git.sh\nindex b105d6d9d5..3f6396ef63 100755\n--- a/t/t9700-perl-git.sh\n+++ b/t/t9700-perl-git.sh\n@@ -13,9 +13,8 @@ skip_all_if_no_Test_More\n \n # set up test repository\n \n-test_expect_success \\\n-    'set up test repository' \\\n-    'echo \"test file 1\" > file1 &&\n+test_expect_success 'set up test repository' '\n+     echo \"test file 1\" > file1 &&\n      echo \"test file 2\" > file2 &&\n      mkdir directory1 &&\n      echo \"in directory1\" >> directory1/file &&\n-- \n2.29.0-rc0\n\n"},{"id":"472429","messageId":"20230222040745.1511205-3-18994118902@163.com","threadId":"59285","inReplyTo":"20230222040745.1511205-1-18994118902@163.com","subject":"[GSOC] [PATCH v1 2/2] Change indents to TAB","fromName":"Zhang Yi","fromEmail":"18994118902@163.com","sentAt":"2023-02-22T04:07:45Z","receivedAt":"2023-02-22T04:08:39Z","isPatch":true,"sender":{"key":"18994118902@163.com","avatar":"https://avatars.githubusercontent.com/u/48300302?v=4"},"body":"Signed-off-by: Zhang Yi <18994118902@163.com>\n---\n t/t9700-perl-git.sh | 58 ++++++++++++++++++++++-----------------------\n 1 file changed, 29 insertions(+), 29 deletions(-)\n\ndiff --git a/t/t9700-perl-git.sh b/t/t9700-perl-git.sh\nindex 3f6396ef63..671875a3fc 100755\n--- a/t/t9700-perl-git.sh\n+++ b/t/t9700-perl-git.sh\n@@ -14,35 +14,35 @@ skip_all_if_no_Test_More\n # set up test repository\n \n test_expect_success 'set up test repository' '\n-     echo \"test file 1\" > file1 &&\n-     echo \"test file 2\" > file2 &&\n-     mkdir directory1 &&\n-     echo \"in directory1\" >> directory1/file &&\n-     mkdir directory2 &&\n-     echo \"in directory2\" >> directory2/file &&\n-     git add . &&\n-     git commit -m \"first commit\" &&\n-\n-     echo \"new file in subdir 2\" > directory2/file2 &&\n-     git add . &&\n-     git commit -m \"commit in directory2\" &&\n-\n-     echo \"changed file 1\" > file1 &&\n-     git commit -a -m \"second commit\" &&\n-\n-     git config --add color.test.slot1 green &&\n-     git config --add test.string value &&\n-     git config --add test.dupstring value1 &&\n-     git config --add test.dupstring value2 &&\n-     git config --add test.booltrue true &&\n-     git config --add test.boolfalse no &&\n-     git config --add test.boolother other &&\n-     git config --add test.int 2k &&\n-     git config --add test.path \"~/foo\" &&\n-     git config --add test.pathexpanded \"$HOME/foo\" &&\n-     git config --add test.pathmulti foo &&\n-     git config --add test.pathmulti bar\n-     '\n+\techo \"test file 1\" > file1 &&\n+\techo \"test file 2\" > file2 &&\n+\tmkdir directory1 &&\n+\techo \"in directory1\" >> directory1/file &&\n+\tmkdir directory2 &&\n+\techo \"in directory2\" >> directory2/file &&\n+\tgit add . &&\n+\tgit commit -m \"first commit\" &&\n+\n+\techo \"new file in subdir 2\" > directory2/file2 &&\n+\tgit add . &&\n+\tgit commit -m \"commit in directory2\" &&\n+\n+\techo \"changed file 1\" > file1 &&\n+\tgit commit -a -m \"second commit\" &&\n+\n+\tgit config --add color.test.slot1 green &&\n+\tgit config --add test.string value &&\n+\tgit config --add test.dupstring value1 &&\n+\tgit config --add test.dupstring value2 &&\n+\tgit config --add test.booltrue true &&\n+\tgit config --add test.boolfalse no &&\n+\tgit config --add test.boolother other &&\n+\tgit config --add test.int 2k &&\n+\tgit config --add test.path \"~/foo\" &&\n+\tgit config --add test.pathexpanded \"$HOME/foo\" &&\n+\tgit config --add test.pathmulti foo &&\n+\tgit config --add test.pathmulti bar\n+'\n \n test_expect_success 'set up bare repository' '\n \tgit init --bare bare.git\n-- \n2.29.0-rc0\n\n"},{"id":"472434","messageId":"CADupsJPpZnjA=Pu_RZZZXy7Titj3UD7ppww48KvcHHHbrGx=rw@mail.gmail.com","threadId":"59285","inReplyTo":"20230222040745.1511205-1-18994118902@163.com","subject":"Re: [GSOC] [PATCH v1 0/2] Modernize t9700 test script","fromName":"Vivan Garg","fromEmail":"v.garg.work@gmail.com","sentAt":"2023-02-22T07:09:13Z","receivedAt":"2023-02-22T07:09:30Z","isPatch":true,"sender":{"key":"v.garg.work@gmail.com","avatar":null},"body":"Hi Zhang Yi,\n\n> Subject: [GSOC] [PATCH v1 0/2] Modernize t9700 test script\n\nIt is the convention here to start the subject line with the name of the\ncomponent you're working on, and it's always good to follow the\nconvention. Something like:\n\nt9700: modernize test style\n\nNote that the part after the colon should start with a lowercase letter.\n\n> A title in the test script is in old style an a body is indented by spaces.I fix the title style and change indents to TAB.\n>\n> Zhang Yi (2):\n>   Fix title style\n>   Change indents  to TAB\n>\n>  t/t9700-perl-git.sh | 61 ++++++++++++++++++++++-----------------------\n>  1 file changed, 30 insertions(+), 31 deletions(-)\n>\n> --\n> 2.29.0-rc0\n>\n"},{"id":"472435","messageId":"CADupsJMW8fRDvhknxFgnSRQ4O7+YDHkKsEBoFq-yhO2vCq5MBg@mail.gmail.com","threadId":"59285","inReplyTo":"20230222040745.1511205-2-18994118902@163.com","subject":"Re: [GSOC] [PATCH v1 1/2] Fix title style","fromName":"Vivan Garg","fromEmail":"v.garg.work@gmail.com","sentAt":"2023-02-22T07:13:18Z","receivedAt":"2023-02-22T07:13:33Z","isPatch":true,"sender":{"key":"v.garg.work@gmail.com","avatar":null},"body":"You are missing the body of the commit message. Commit message\nshould start with description of the existing problem in present tense,\nsomething like:\n\n     Test scripts in file t9700-perl-git.sh are written in old style,\n     where the test_expect_success command and test title are written on\n     separate lines ...\n\nThen changes should be described using imperative mood, as if you are\ngiving commands to the codebase.  See section \"[[describe-changes]]\"\nin \"Documentation/SubmittingPatches\" for details.\n\n> Signed-off-by: Zhang Yi <18994118902@163.com>\n> ---\n>  t/t9700-perl-git.sh | 5 ++---\n>  1 file changed, 2 insertions(+), 3 deletions(-)\n>\n> diff --git a/t/t9700-perl-git.sh b/t/t9700-perl-git.sh\n> index b105d6d9d5..3f6396ef63 100755\n> --- a/t/t9700-perl-git.sh\n> +++ b/t/t9700-perl-git.sh\n> @@ -13,9 +13,8 @@ skip_all_if_no_Test_More\n>\n>  # set up test repository\n>\n> -test_expect_success \\\n> -    'set up test repository' \\\n> -    'echo \"test file 1\" > file1 &&\n> +test_expect_success 'set up test repository' '\n> +     echo \"test file 1\" > file1 &&\n>       echo \"test file 2\" > file2 &&\n>       mkdir directory1 &&\n>       echo \"in directory1\" >> directory1/file &&\n> --\n\nthere should be no whitespace after redirect operators, so:\n\n    echo \"test file 1\" > file1 &&\n\n  should become:\n\n    echo \"test file 1\" >file1 &&\n\nSimilar changes for the rest of the test script as well.\n\n> 2.29.0-rc0\n>\n"},{"id":"472436","messageId":"CADupsJPLF94EnMHYNO08dFV+DEooeB6+YB8bgkRJwBrFtBbOwQ@mail.gmail.com","threadId":"59285","inReplyTo":"20230222040745.1511205-3-18994118902@163.com","subject":"Re: [GSOC] [PATCH v1 2/2] Change indents to TAB","fromName":"Vivan Garg","fromEmail":"v.garg.work@gmail.com","sentAt":"2023-02-22T07:24:15Z","receivedAt":"2023-02-22T07:24:32Z","isPatch":true,"sender":{"key":"v.garg.work@gmail.com","avatar":null},"body":"You are missing the body of the commit message here as well, Please add one\nin your next re-roll.\n\nOn Tue, Feb 21, 2023 at 10:01 PM Zhang Yi <18994118902@163.com> wrote:\n>\n> Signed-off-by: Zhang Yi <18994118902@163.com>\n> ---\n>  t/t9700-perl-git.sh | 58 ++++++++++++++++++++++-----------------------\n>  1 file changed, 29 insertions(+), 29 deletions(-)\n>\n> diff --git a/t/t9700-perl-git.sh b/t/t9700-perl-git.sh\n> index 3f6396ef63..671875a3fc 100755\n> --- a/t/t9700-perl-git.sh\n> +++ b/t/t9700-perl-git.sh\n> @@ -14,35 +14,35 @@ skip_all_if_no_Test_More\n>  # set up test repository\n>\n>  test_expect_success 'set up test repository' '\n> -     echo \"test file 1\" > file1 &&\n> -     echo \"test file 2\" > file2 &&\n> -     mkdir directory1 &&\n> -     echo \"in directory1\" >> directory1/file &&\n> -     mkdir directory2 &&\n> -     echo \"in directory2\" >> directory2/file &&\n> -     git add . &&\n> -     git commit -m \"first commit\" &&\n> -\n> -     echo \"new file in subdir 2\" > directory2/file2 &&\n> -     git add . &&\n> -     git commit -m \"commit in directory2\" &&\n> -\n> -     echo \"changed file 1\" > file1 &&\n> -     git commit -a -m \"second commit\" &&\n> -\n> -     git config --add color.test.slot1 green &&\n> -     git config --add test.string value &&\n> -     git config --add test.dupstring value1 &&\n> -     git config --add test.dupstring value2 &&\n> -     git config --add test.booltrue true &&\n> -     git config --add test.boolfalse no &&\n> -     git config --add test.boolother other &&\n> -     git config --add test.int 2k &&\n> -     git config --add test.path \"~/foo\" &&\n> -     git config --add test.pathexpanded \"$HOME/foo\" &&\n> -     git config --add test.pathmulti foo &&\n> -     git config --add test.pathmulti bar\n> -     '\n> +       echo \"test file 1\" > file1 &&\n> +       echo \"test file 2\" > file2 &&\n> +       mkdir directory1 &&\n> +       echo \"in directory1\" >> directory1/file &&\n> +       mkdir directory2 &&\n> +       echo \"in directory2\" >> directory2/file &&\n> +       git add . &&\n> +       git commit -m \"first commit\" &&\n> +\n> +       echo \"new file in subdir 2\" > directory2/file2 &&\n> +       git add . &&\n> +       git commit -m \"commit in directory2\" &&\n> +\n> +       echo \"changed file 1\" > file1 &&\n> +       git commit -a -m \"second commit\" &&\n> +\n> +       git config --add color.test.slot1 green &&\n> +       git config --add test.string value &&\n> +       git config --add test.dupstring value1 &&\n> +       git config --add test.dupstring value2 &&\n> +       git config --add test.booltrue true &&\n> +       git config --add test.boolfalse no &&\n> +       git config --add test.boolother other &&\n> +       git config --add test.int 2k &&\n> +       git config --add test.path \"~/foo\" &&\n> +       git config --add test.pathexpanded \"$HOME/foo\" &&\n> +       git config --add test.pathmulti foo &&\n> +       git config --add test.pathmulti bar\n> +'\n>\n>  test_expect_success 'set up bare repository' '\n>         git init --bare bare.git\n> --\n\nThere should be no whitespace after redirect operators; as described in the last\ncommit of this patch. Make sure to fix that in your next re-roll for\nthis test script\nas well. You might also want to look at this for a summary of all the\nthings you\nneed to do:\nhttps://lore.kernel.org/git/CAPig+cQpUu2UO-+jWn1nTaDykWnxwuEitzVB7PnW2SS_b7V8Hg@mail.gmail.com/\n\n> 2.29.0-rc0\n>\n\nThanks!\n"},{"id":"472606","messageId":"20230224013413.1969003-1-18994118902@163.com","threadId":"59285","inReplyTo":"CADupsJPpZnjA=Pu_RZZZXy7Titj3UD7ppww48KvcHHHbrGx=rw@mail.gmail.com","subject":"[GSOC] [PATCH v2 0/3] t9700:moderizen test scripts","fromName":"Zhang Yi","fromEmail":"18994118902@163.com","sentAt":"2023-02-24T01:34:10Z","receivedAt":"2023-02-24T01:37:00Z","isPatch":true,"sender":{"key":"18994118902@163.com","avatar":"https://avatars.githubusercontent.com/u/48300302?v=4"},"body":"The style of t9700-perl-git.sh is old.There are 3 things need improve:\n* A title is not on the same line with test_expect_success command.Need to combine them.\n* A test body is indented by whitespaces.Need to replace whitespaces by TAB\n* There are whitespaces after redirect operators.Need to delete whitespaces.\n\n---\nThanks for all suggestions.Here are so many interesting things to learn.\n\nZhang Yi (3):\n  t9700:fix title style\n  t9700:change indents to TAB\n  t9700:delete whitespaces after redirect operators\n\n t/t9700-perl-git.sh | 61 ++++++++++++++++++++++-----------------------\n 1 file changed, 30 insertions(+), 31 deletions(-)\n\n-- \n2.29.0-rc0\n\n"},{"id":"472607","messageId":"20230224013413.1969003-2-18994118902@163.com","threadId":"59285","inReplyTo":"20230224013413.1969003-1-18994118902@163.com","subject":"[GSOC] [PATCH v2 1/3] t9700:fix title style","fromName":"Zhang Yi","fromEmail":"18994118902@163.com","sentAt":"2023-02-24T01:34:11Z","receivedAt":"2023-02-24T01:37:04Z","isPatch":true,"sender":{"key":"18994118902@163.com","avatar":"https://avatars.githubusercontent.com/u/48300302?v=4"},"body":"One title in t9700-perl-git.sh is in old style where the\ntest_expect_success command and test title are written on\nseparate lines.\n\nCombine the test_expect_success command and the title in one line.\n\nSigned-off-by: Zhang Yi <18994118902@163.com>\n---\n t/t9700-perl-git.sh | 5 ++---\n 1 file changed, 2 insertions(+), 3 deletions(-)\n\ndiff --git a/t/t9700-perl-git.sh b/t/t9700-perl-git.sh\nindex b105d6d9d5..3f6396ef63 100755\n--- a/t/t9700-perl-git.sh\n+++ b/t/t9700-perl-git.sh\n@@ -13,9 +13,8 @@ skip_all_if_no_Test_More\n \n # set up test repository\n \n-test_expect_success \\\n-    'set up test repository' \\\n-    'echo \"test file 1\" > file1 &&\n+test_expect_success 'set up test repository' '\n+     echo \"test file 1\" > file1 &&\n      echo \"test file 2\" > file2 &&\n      mkdir directory1 &&\n      echo \"in directory1\" >> directory1/file &&\n-- \n2.29.0-rc0\n\n"},{"id":"472608","messageId":"20230224013413.1969003-3-18994118902@163.com","threadId":"59285","inReplyTo":"20230224013413.1969003-1-18994118902@163.com","subject":"[GSOC] [PATCH v2 2/3] t9700:change indents to TAB","fromName":"Zhang Yi","fromEmail":"18994118902@163.com","sentAt":"2023-02-24T01:34:12Z","receivedAt":"2023-02-24T01:37:06Z","isPatch":true,"sender":{"key":"18994118902@163.com","avatar":"https://avatars.githubusercontent.com/u/48300302?v=4"},"body":"The body of one test script in t9700-perl-git.sh is indented by spaces\nwhile others are indented by TAB.And test bodies should be indented with\nTAB, not spaces.\n\nReplace all space indents by TAB.\n\nSigned-off-by: Zhang Yi <18994118902@163.com>\n---\n t/t9700-perl-git.sh | 58 ++++++++++++++++++++++-----------------------\n 1 file changed, 29 insertions(+), 29 deletions(-)\n\ndiff --git a/t/t9700-perl-git.sh b/t/t9700-perl-git.sh\nindex 3f6396ef63..671875a3fc 100755\n--- a/t/t9700-perl-git.sh\n+++ b/t/t9700-perl-git.sh\n@@ -14,35 +14,35 @@ skip_all_if_no_Test_More\n # set up test repository\n \n test_expect_success 'set up test repository' '\n-     echo \"test file 1\" > file1 &&\n-     echo \"test file 2\" > file2 &&\n-     mkdir directory1 &&\n-     echo \"in directory1\" >> directory1/file &&\n-     mkdir directory2 &&\n-     echo \"in directory2\" >> directory2/file &&\n-     git add . &&\n-     git commit -m \"first commit\" &&\n-\n-     echo \"new file in subdir 2\" > directory2/file2 &&\n-     git add . &&\n-     git commit -m \"commit in directory2\" &&\n-\n-     echo \"changed file 1\" > file1 &&\n-     git commit -a -m \"second commit\" &&\n-\n-     git config --add color.test.slot1 green &&\n-     git config --add test.string value &&\n-     git config --add test.dupstring value1 &&\n-     git config --add test.dupstring value2 &&\n-     git config --add test.booltrue true &&\n-     git config --add test.boolfalse no &&\n-     git config --add test.boolother other &&\n-     git config --add test.int 2k &&\n-     git config --add test.path \"~/foo\" &&\n-     git config --add test.pathexpanded \"$HOME/foo\" &&\n-     git config --add test.pathmulti foo &&\n-     git config --add test.pathmulti bar\n-     '\n+\techo \"test file 1\" > file1 &&\n+\techo \"test file 2\" > file2 &&\n+\tmkdir directory1 &&\n+\techo \"in directory1\" >> directory1/file &&\n+\tmkdir directory2 &&\n+\techo \"in directory2\" >> directory2/file &&\n+\tgit add . &&\n+\tgit commit -m \"first commit\" &&\n+\n+\techo \"new file in subdir 2\" > directory2/file2 &&\n+\tgit add . &&\n+\tgit commit -m \"commit in directory2\" &&\n+\n+\techo \"changed file 1\" > file1 &&\n+\tgit commit -a -m \"second commit\" &&\n+\n+\tgit config --add color.test.slot1 green &&\n+\tgit config --add test.string value &&\n+\tgit config --add test.dupstring value1 &&\n+\tgit config --add test.dupstring value2 &&\n+\tgit config --add test.booltrue true &&\n+\tgit config --add test.boolfalse no &&\n+\tgit config --add test.boolother other &&\n+\tgit config --add test.int 2k &&\n+\tgit config --add test.path \"~/foo\" &&\n+\tgit config --add test.pathexpanded \"$HOME/foo\" &&\n+\tgit config --add test.pathmulti foo &&\n+\tgit config --add test.pathmulti bar\n+'\n \n test_expect_success 'set up bare repository' '\n \tgit init --bare bare.git\n-- \n2.29.0-rc0\n\n"},{"id":"472609","messageId":"20230224013413.1969003-4-18994118902@163.com","threadId":"59285","inReplyTo":"20230224013413.1969003-1-18994118902@163.com","subject":"[GSOC] [PATCH v2 3/3] t9700:delete whitespaces after redirect operators","fromName":"Zhang Yi","fromEmail":"18994118902@163.com","sentAt":"2023-02-24T01:34:13Z","receivedAt":"2023-02-24T01:37:09Z","isPatch":true,"sender":{"key":"18994118902@163.com","avatar":"https://avatars.githubusercontent.com/u/48300302?v=4"},"body":"There are whitespaces after redirect operators,which is a deprecated\nstyle.\n\nDelete whitespaces after redirect operators.\n\nSigned-off-by: Zhang Yi <18994118902@163.com>\n---\n t/t9700-perl-git.sh | 12 ++++++------\n 1 file changed, 6 insertions(+), 6 deletions(-)\n\ndiff --git a/t/t9700-perl-git.sh b/t/t9700-perl-git.sh\nindex 671875a3fc..ccc8212d73 100755\n--- a/t/t9700-perl-git.sh\n+++ b/t/t9700-perl-git.sh\n@@ -14,20 +14,20 @@ skip_all_if_no_Test_More\n # set up test repository\n \n test_expect_success 'set up test repository' '\n-\techo \"test file 1\" > file1 &&\n-\techo \"test file 2\" > file2 &&\n+\techo \"test file 1\" >file1 &&\n+\techo \"test file 2\" >file2 &&\n \tmkdir directory1 &&\n-\techo \"in directory1\" >> directory1/file &&\n+\techo \"in directory1\" >>directory1/file &&\n \tmkdir directory2 &&\n-\techo \"in directory2\" >> directory2/file &&\n+\techo \"in directory2\" >>directory2/file &&\n \tgit add . &&\n \tgit commit -m \"first commit\" &&\n \n-\techo \"new file in subdir 2\" > directory2/file2 &&\n+\techo \"new file in subdir 2\" >directory2/file2 &&\n \tgit add . &&\n \tgit commit -m \"commit in directory2\" &&\n \n-\techo \"changed file 1\" > file1 &&\n+\techo \"changed file 1\" >file1 &&\n \tgit commit -a -m \"second commit\" &&\n \n \tgit config --add color.test.slot1 green &&\n-- \n2.29.0-rc0\n\n"},{"id":"472612","messageId":"xmqqo7pj3hi7.fsf@gitster.g","threadId":"59285","inReplyTo":"20230224013413.1969003-1-18994118902@163.com","subject":"Re: [GSOC] [PATCH v2 0/3] t9700:moderizen test scripts","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2023-02-24T02:49:04Z","receivedAt":"2023-02-24T02:49:12Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Zhang Yi <18994118902@163.com> writes:\n\n> The style of t9700-perl-git.sh is old.There are 3 things need improve:\n> * A title is not on the same line with test_expect_success command.Need to combine them.\n> * A test body is indented by whitespaces.Need to replace whitespaces by TAB\n> * There are whitespaces after redirect operators.Need to delete whitespaces.\n>\n> ---\n> Thanks for all suggestions.Here are so many interesting things to learn.\n\nIt is a good idea to summarize the difference between the previous\nround and this round, to help reviewers who have already spent\ncycles to review the previous round to skim the parts of the updated\nseries that haven't been changed (and they have already reviewed).\n\nAnd the cover letter is the ideal place to do so.\n\nAlternatively, you can write after the three-dash lines between the\nproposed log message and the patch text of each patch message.\n\n> Zhang Yi (3):\n>   t9700:fix title style\n>   t9700:change indents to TAB\n>   t9700:delete whitespaces after redirect operators\n\nYou are missing space after the colon, all of them.  Also you are\nmissing space after full-stop that concludes each sentence in all of\nyour proposed log messages.\n\nEven though you _could_ treat these three things as logically\nseparate changes, I would suspect that each of them is so small that\nit may be easier to review as a single patch.\n\nThanks.\n"},{"id":"472614","messageId":"33d3469b.2a04.186817a2bf2.Coremail.18994118902@163.com","threadId":"59285","inReplyTo":"xmqqo7pj3hi7.fsf@gitster.g","subject":"Re:Re: [GSOC] [PATCH v2 0/3] t9700:moderizen test scripts","fromName":"Zhang Yi","fromEmail":"18994118902@163.com","sentAt":"2023-02-24T03:31:52Z","receivedAt":"2023-02-24T03:48:26Z","isPatch":true,"sender":{"key":"18994118902@163.com","avatar":"https://avatars.githubusercontent.com/u/48300302?v=4"},"body":"Ok. Thanks.\n\nSo I need to do:\n\n* Summary the update between versions in the cover letter.\n* Add space after colon in patch titles.\n* Add space after full-stop in proposed log messages. \nJust like \"The style of t9700-perl-git.sh is old. There are 3 things need improve\".\n* Combine 3 patches into 1.\n\nHope no misunderstanding. If something wrong, please address me.\n\n"},{"id":"472646","messageId":"33f15034-4271-cc38-8162-1eadab392552@iee.email","threadId":"59285","inReplyTo":"20230224013413.1969003-4-18994118902@163.com","subject":"Re: [GSOC] [PATCH v2 3/3] t9700:delete whitespaces after redirect operators","fromName":"Philip Oakley","fromEmail":"philipoakley@iee.email","sentAt":"2023-02-24T11:20:22Z","receivedAt":"2023-02-24T11:20:33Z","isPatch":true,"sender":{"key":"philipoakley@iee.email","avatar":"https://avatars.githubusercontent.com/u/914343?v=4"},"body":"On 24/02/2023 01:34, Zhang Yi wrote:\n> There are whitespaces after redirect operators,which is a deprecated\n\nmicro-nit: space after comma, before \"which\".\n\nPhilip\n> style.\n>\n> Delete whitespaces after redirect operators.\n>\n> Signed-off-by: Zhang Yi <18994118902@163.com>\n> ---\n>  t/t9700-perl-git.sh | 12 ++++++------\n>  1 file changed, 6 insertions(+), 6 deletions(-)\n>\n> diff --git a/t/t9700-perl-git.sh b/t/t9700-perl-git.sh\n> index 671875a3fc..ccc8212d73 100755\n> --- a/t/t9700-perl-git.sh\n> +++ b/t/t9700-perl-git.sh\n> @@ -14,20 +14,20 @@ skip_all_if_no_Test_More\n>  # set up test repository\n>  \n>  test_expect_success 'set up test repository' '\n> -\techo \"test file 1\" > file1 &&\n> -\techo \"test file 2\" > file2 &&\n> +\techo \"test file 1\" >file1 &&\n> +\techo \"test file 2\" >file2 &&\n>  \tmkdir directory1 &&\n> -\techo \"in directory1\" >> directory1/file &&\n> +\techo \"in directory1\" >>directory1/file &&\n>  \tmkdir directory2 &&\n> -\techo \"in directory2\" >> directory2/file &&\n> +\techo \"in directory2\" >>directory2/file &&\n>  \tgit add . &&\n>  \tgit commit -m \"first commit\" &&\n>  \n> -\techo \"new file in subdir 2\" > directory2/file2 &&\n> +\techo \"new file in subdir 2\" >directory2/file2 &&\n>  \tgit add . &&\n>  \tgit commit -m \"commit in directory2\" &&\n>  \n> -\techo \"changed file 1\" > file1 &&\n> +\techo \"changed file 1\" >file1 &&\n>  \tgit commit -a -m \"second commit\" &&\n>  \n>  \tgit config --add color.test.slot1 green &&\n\n"},{"id":"472696","messageId":"20230225013003.1971931-1-18994118902@163.com","threadId":"59285","inReplyTo":"CADupsJPpZnjA=Pu_RZZZXy7Titj3UD7ppww48KvcHHHbrGx=rw@mail.gmail.com","subject":"[GSOC] [PATCH v3 1/1] t9700: modernize test scripts","fromName":"Zhang Yi","fromEmail":"18994118902@163.com","sentAt":"2023-02-25T01:30:03Z","receivedAt":"2023-02-25T01:33:23Z","isPatch":true,"sender":{"key":"18994118902@163.com","avatar":"https://avatars.githubusercontent.com/u/48300302?v=4"},"body":"The style of t9700-perl-git.sh is old. There are 3 problems:\n* A title is not on the same line with test_expect_success command.\n* A test body is indented by whitespaces.\n* There are whitespaces after redirect operators.\n\nModernize test scripts by:\n* Combine the title with test_expect_success command.\n* Replace whitespace indents with TAB.\n* Delete whitespaces after redirect operators.\n\nSigned-off-by: Zhang Yi <18994118902@163.com>\n---\nUpdate compare to v2:\n* Add summary between versions.\n* Add space after colon in patch title.\n* Add space after full stops and after comma.\n* Combine 3 patches into 1.\n\nAfter review 'First steps contributing to Git', I find it available to\nsend a single update patch for new version. I think this way is eaiser to\nreview. \n\nThanks for help from Junio C Hamano, Philip Oaklcy and again for Vivan\nGarg. \n\n t/t9700-perl-git.sh | 61 ++++++++++++++++++++++-----------------------\n 1 file changed, 30 insertions(+), 31 deletions(-)\n\n\ndiff --git a/t/t9700-perl-git.sh b/t/t9700-perl-git.sh\nindex b105d6d9d5..ccc8212d73 100755\n--- a/t/t9700-perl-git.sh\n+++ b/t/t9700-perl-git.sh\n@@ -13,37 +13,36 @@ skip_all_if_no_Test_More\n \n # set up test repository\n \n-test_expect_success \\\n-    'set up test repository' \\\n-    'echo \"test file 1\" > file1 &&\n-     echo \"test file 2\" > file2 &&\n-     mkdir directory1 &&\n-     echo \"in directory1\" >> directory1/file &&\n-     mkdir directory2 &&\n-     echo \"in directory2\" >> directory2/file &&\n-     git add . &&\n-     git commit -m \"first commit\" &&\n-\n-     echo \"new file in subdir 2\" > directory2/file2 &&\n-     git add . &&\n-     git commit -m \"commit in directory2\" &&\n-\n-     echo \"changed file 1\" > file1 &&\n-     git commit -a -m \"second commit\" &&\n-\n-     git config --add color.test.slot1 green &&\n-     git config --add test.string value &&\n-     git config --add test.dupstring value1 &&\n-     git config --add test.dupstring value2 &&\n-     git config --add test.booltrue true &&\n-     git config --add test.boolfalse no &&\n-     git config --add test.boolother other &&\n-     git config --add test.int 2k &&\n-     git config --add test.path \"~/foo\" &&\n-     git config --add test.pathexpanded \"$HOME/foo\" &&\n-     git config --add test.pathmulti foo &&\n-     git config --add test.pathmulti bar\n-     '\n+test_expect_success 'set up test repository' '\n+\techo \"test file 1\" >file1 &&\n+\techo \"test file 2\" >file2 &&\n+\tmkdir directory1 &&\n+\techo \"in directory1\" >>directory1/file &&\n+\tmkdir directory2 &&\n+\techo \"in directory2\" >>directory2/file &&\n+\tgit add . &&\n+\tgit commit -m \"first commit\" &&\n+\n+\techo \"new file in subdir 2\" >directory2/file2 &&\n+\tgit add . &&\n+\tgit commit -m \"commit in directory2\" &&\n+\n+\techo \"changed file 1\" >file1 &&\n+\tgit commit -a -m \"second commit\" &&\n+\n+\tgit config --add color.test.slot1 green &&\n+\tgit config --add test.string value &&\n+\tgit config --add test.dupstring value1 &&\n+\tgit config --add test.dupstring value2 &&\n+\tgit config --add test.booltrue true &&\n+\tgit config --add test.boolfalse no &&\n+\tgit config --add test.boolother other &&\n+\tgit config --add test.int 2k &&\n+\tgit config --add test.path \"~/foo\" &&\n+\tgit config --add test.pathexpanded \"$HOME/foo\" &&\n+\tgit config --add test.pathmulti foo &&\n+\tgit config --add test.pathmulti bar\n+'\n \n test_expect_success 'set up bare repository' '\n \tgit init --bare bare.git\n-- \n2.29.0-rc0\n\n"}]}