{"thread":{"id":"46770","subject":"[PATCH] t2018: remove unwanted empty line","startedAt":"2017-09-17T10:19:33Z","lastAt":"2017-09-18T13:38:10Z","messageCount":3,"participants":["Kaartic Sivaraam","Kevin Daudt"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"328258","messageId":"0102015e8f599df4-6e3b45f3-b3ea-4483-ad62-08119e29b1cd-000000@eu-west-1.amazonses.com","threadId":"46770","inReplyTo":null,"subject":"[PATCH] t2018: remove unwanted empty line","fromName":"Kaartic Sivaraam","fromEmail":"kaarticsivaraam91196@gmail.com","sentAt":"2017-09-17T10:19:28Z","receivedAt":"2017-09-17T10:19:33Z","isPatch":true,"sender":{"key":"kaartic.sivaraam@gmail.com","avatar":"https://avatars.githubusercontent.com/u/12448084?v=4"},"body":"Signed-off-by: Kaartic Sivaraam <kaarticsivaraam91196@gmail.com>\n---\n t/t2018-checkout-branch.sh | 1 -\n 1 file changed, 1 deletion(-)\n\ndiff --git a/t/t2018-checkout-branch.sh b/t/t2018-checkout-branch.sh\nindex 2131fb2a5682c..e0a52334a22dd 100755\n--- a/t/t2018-checkout-branch.sh\n+++ b/t/t2018-checkout-branch.sh\n@@ -192,7 +192,6 @@ test_expect_success 'checkout -b <describe>' '\n test_expect_success 'checkout -B to the current branch works' '\n \tgit checkout branch1 &&\n \tgit checkout -B branch1-scratch &&\n-\n \tsetup_dirty_mergeable &&\n \tgit checkout -B branch1-scratch initial &&\n \ttest_dirty_mergeable\n\n--\nhttps://github.com/git/git/pull/403\n"},{"id":"328268","messageId":"20170917192227.GD21499@alpha.vpn.ikke.info","threadId":"46770","inReplyTo":"0102015e8f599df4-6e3b45f3-b3ea-4483-ad62-08119e29b1cd-000000@eu-west-1.amazonses.com","subject":"Re: [PATCH] t2018: remove unwanted empty line","fromName":"Kevin Daudt","fromEmail":"me@ikke.info","sentAt":"2017-09-17T19:22:27Z","receivedAt":"2017-09-17T19:40:23Z","isPatch":true,"sender":{"key":"me@ikke.info","avatar":"https://avatars.githubusercontent.com/u/135698?v=4"},"body":"On Sun, Sep 17, 2017 at 10:19:28AM +0000, Kaartic Sivaraam wrote:\n> Signed-off-by: Kaartic Sivaraam <kaarticsivaraam91196@gmail.com>\n> ---\n>  t/t2018-checkout-branch.sh | 1 -\n>  1 file changed, 1 deletion(-)\n> \n> diff --git a/t/t2018-checkout-branch.sh b/t/t2018-checkout-branch.sh\n> index 2131fb2a5682c..e0a52334a22dd 100755\n> --- a/t/t2018-checkout-branch.sh\n> +++ b/t/t2018-checkout-branch.sh\n> @@ -192,7 +192,6 @@ test_expect_success 'checkout -b <describe>' '\n>  test_expect_success 'checkout -B to the current branch works' '\n>  \tgit checkout branch1 &&\n>  \tgit checkout -B branch1-scratch &&\n> -\n>  \tsetup_dirty_mergeable &&\n>  \tgit checkout -B branch1-scratch initial &&\n>  \ttest_dirty_mergeable\n> \n> --\n> https://github.com/git/git/pull/403\n\nWhy is this empty line unwanted? This kind of whitespace can help\nseparate logical sections, just like paragraphs would.\n\nKevin.\n"},{"id":"328290","messageId":"e7fd5df4-4496-09a2-5f67-7ae6868bb3e8@gmail.com","threadId":"46770","inReplyTo":"20170917192227.GD21499@alpha.vpn.ikke.info","subject":"Re: [PATCH] t2018: remove unwanted empty line","fromName":"Kaartic Sivaraam","fromEmail":"kaarticsivaraam91196@gmail.com","sentAt":"2017-09-18T13:37:58Z","receivedAt":"2017-09-18T13:38:10Z","isPatch":true,"sender":{"key":"kaartic.sivaraam@gmail.com","avatar":"https://avatars.githubusercontent.com/u/12448084?v=4"},"body":"On Monday 18 September 2017 12:52 AM, Kevin Daudt wrote:\n> Why is this empty line unwanted? This kind of whitespace can help\n> separate logical sections, just like paragraphs would.\n>\nYou're right. I seem to have sent a fix precariously because I haven't \nsuch separation\nbefore (forgetting the fact that git has contributors whose way of \nwriting test vary diversly).\nShould better stop back and think the next time rather than spamming the \nlist. Sorry, for this.\n\n---\nKaartic\n"}]}