[PATCH 2/6] t5516 (fetch-push): update test description

2013-04-02 Thread Ramkumar Ramachandra
The file was originally created in bcdb34f (Test wildcard push/fetch, 2007-06-08), and only contained tests that exercised wildcard functionality at the time. In subsequent commits, many other tests unrelated to wildcards were added but the test description was never updated. Fix this. Helped-by

[PATCH 2/6] t5516 (fetch-push): update test description

2013-03-28 Thread Ramkumar Ramachandra
The file was originally created in bcdb34f (Test wildcard push/fetch, 2007-06-08), and only contained tests that exercised wildcard functionality at the time. In subsequent commits, many other tests unrelated to wildcards were added but the test description was never updated. Fix this. Helped-by

Re: [PATCH 2/6] t5516 (fetch-push): update test description

2013-03-22 Thread Jonathan Nieder
Ramkumar Ramachandra wrote: > --- a/t/t5516-fetch-push.sh > +++ b/t/t5516-fetch-push.sh > @@ -1,6 +1,6 @@ > #!/bin/sh > > -test_description='fetching and pushing, with or without wildcard' > +test_description='fetching and pushing' The description before and after are equally useless. You mig

[PATCH 2/6] t5516 (fetch-push): update test description

2013-03-22 Thread Ramkumar Ramachandra
The file was originally created in bcdb34f (Test wildcard push/fetch, 2007-06-08), and only contained tests that exercised wildcard functionality at the time. In subsequent commits, many other tests unrelated to wildcards were added but the test description was never updated. Fix this. Signed-of

Re: [PATCH 2/6] t5516 (fetch-push): update test description

2013-03-20 Thread Jonathan Nieder
Ramkumar Ramachandra wrote: > When I want to add a test for branch..pushremote, I grep > for branch.*.pushurl, and open files with sensible names; I'm not > going to open up the file and read a long description of what tests it > already contains. Huh? The test_description is output for "./

Re: [PATCH 2/6] t5516 (fetch-push): update test description

2013-03-20 Thread Ramkumar Ramachandra
Jonathan Nieder wrote: > Ramkumar Ramachandra wrote: > >> --- a/t/t5516-fetch-push.sh >> +++ b/t/t5516-fetch-push.sh >> @@ -1,6 +1,6 @@ >> #!/bin/sh >> >> -test_description='fetching and pushing, with or without wildcard' >> +test_description='fetching and pushing' > > I'm not thrilled with the de

Re: [PATCH 2/6] t5516 (fetch-push): update test description

2013-03-20 Thread Jonathan Nieder
Ramkumar Ramachandra wrote: > --- a/t/t5516-fetch-push.sh > +++ b/t/t5516-fetch-push.sh > @@ -1,6 +1,6 @@ > #!/bin/sh > > -test_description='fetching and pushing, with or without wildcard' > +test_description='fetching and pushing' I'm not thrilled with the description before or after. Would

[PATCH 2/6] t5516 (fetch-push): update test description

2013-03-20 Thread Ramkumar Ramachandra
The file was originally created in bcdb34f (Test wildcard push/fetch, 2007-06-08), and only contained tests that exercised wildcard functionality at the time. In subsequent commits, many other tests unrelated to wildcards were added but the test description was never updated. Fix this. Signed-of