Re: [PATCHv2 2/2] tests for git alternate command

On Thu, Mar 25, 2010 at 12:38 AM, Johannes Sixt [off-list ref] wrote:

Am 3/25/2010 7:07, schrieb Chris Packham:

quoted

Signed-off-by: Chris Packham <redacted>

Your patches violate whitespace rules. Use 'git show --check' to see the
questionable lines.

Will do. BTW theres no mention of --check in git show --help, but one
itch at a time.

quoted

---
I wasn't sure about the test numbering so I just grabbed the highest one. Still
need to add tests for the deletion use case.

According to t/README, t1* would be a suitable category, perhaps t1430.

quoted

 t/t9800-git-alternate.sh |   95 ++++++++++++++++++++++++++++++++++++++++++++++

"git" in the name is redundant

Is t1430-alternate-cmd.sh OK with everyone?

quoted

+test_expect_success \
+     'Setup for rest of the test' '

The modern style for test headers is

[snip]

Will incorporate in next update

quoted

+#rm -rf A B C D base
\ No newline at end of file

test_done

Ah, I knew it would be something simple like that. I guess I should
have started by reading t/README.