# [PATCH] t2018: remove unwanted empty line

3 messages from 2017-09-17 to 2017-09-18. Participants: Kaartic Sivaraam, Kevin Daudt.
Thread: https://gitlist.dev/t/46770

## Kaartic Sivaraam, 2017-09-17 10:19

Subject: [PATCH] t2018: remove unwanted empty line
Message-ID: <0102015e8f599df4-6e3b45f3-b3ea-4483-ad62-08119e29b1cd-000000@eu-west-1.amazonses.com>
URL: https://gitlist.dev/e/0102015e8f599df4-6e3b45f3-b3ea-4483-ad62-08119e29b1cd-000000%40eu-west-1.amazonses.com

```
Signed-off-by: Kaartic Sivaraam <kaarticsivaraam91196@gmail.com>
---
 t/t2018-checkout-branch.sh | 1 -
 1 file changed, 1 deletion(-)

diff --git a/t/t2018-checkout-branch.sh b/t/t2018-checkout-branch.sh
index 2131fb2a5682c..e0a52334a22dd 100755
--- a/t/t2018-checkout-branch.sh
+++ b/t/t2018-checkout-branch.sh
@@ -192,7 +192,6 @@ test_expect_success 'checkout -b <describe>' '
 test_expect_success 'checkout -B to the current branch works' '
 	git checkout branch1 &&
 	git checkout -B branch1-scratch &&
-
 	setup_dirty_mergeable &&
 	git checkout -B branch1-scratch initial &&
 	test_dirty_mergeable

--
https://github.com/git/git/pull/403

```

## Kevin Daudt, 2017-09-17 19:22

Subject: Re: [PATCH] t2018: remove unwanted empty line
Message-ID: <20170917192227.GD21499@alpha.vpn.ikke.info>
URL: https://gitlist.dev/e/20170917192227.GD21499%40alpha.vpn.ikke.info
In-Reply-To: <0102015e8f599df4-6e3b45f3-b3ea-4483-ad62-08119e29b1cd-000000@eu-west-1.amazonses.com>

```
On Sun, Sep 17, 2017 at 10:19:28AM +0000, Kaartic Sivaraam wrote:
> Signed-off-by: Kaartic Sivaraam <kaarticsivaraam91196@gmail.com>
> ---
>  t/t2018-checkout-branch.sh | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/t/t2018-checkout-branch.sh b/t/t2018-checkout-branch.sh
> index 2131fb2a5682c..e0a52334a22dd 100755
> --- a/t/t2018-checkout-branch.sh
> +++ b/t/t2018-checkout-branch.sh
> @@ -192,7 +192,6 @@ test_expect_success 'checkout -b <describe>' '
>  test_expect_success 'checkout -B to the current branch works' '
>  	git checkout branch1 &&
>  	git checkout -B branch1-scratch &&
> -
>  	setup_dirty_mergeable &&
>  	git checkout -B branch1-scratch initial &&
>  	test_dirty_mergeable
> 
> --
> https://github.com/git/git/pull/403

Why is this empty line unwanted? This kind of whitespace can help
separate logical sections, just like paragraphs would.

Kevin.

```

## Kaartic Sivaraam, 2017-09-18 13:37

Subject: Re: [PATCH] t2018: remove unwanted empty line
Message-ID: <e7fd5df4-4496-09a2-5f67-7ae6868bb3e8@gmail.com>
URL: https://gitlist.dev/e/e7fd5df4-4496-09a2-5f67-7ae6868bb3e8%40gmail.com
In-Reply-To: <20170917192227.GD21499@alpha.vpn.ikke.info>

```
On Monday 18 September 2017 12:52 AM, Kevin Daudt wrote:
> Why is this empty line unwanted? This kind of whitespace can help
> separate logical sections, just like paragraphs would.
>
You're right. I seem to have sent a fix precariously because I haven't 
such separation
before (forgetting the fact that git has contributors whose way of 
writing test vary diversly).
Should better stop back and think the next time rather than spamming the 
list. Sorry, for this.

---
Kaartic

```