Re: [PATCH v2] tests: add initial bash completion tests

Felipe Contreras [off-list ref] writes:

On Thu, Apr 12, 2012 at 8:43 PM, Junio C Hamano [off-list ref] wrote:

quoted

Felipe Contreras [off-list ref] writes:

quoted

On Thu, Apr 12, 2012 at 12:57 AM, Felipe Contreras
[off-list ref] wrote:

quoted

+test_expect_success 'double dash' '
+       cat >expected <<-\EOF &&
+       --paginate
+       --no-pager
+       --git-dir=
+       --bare
+       --version
+       --exec-path
+       --html-path
+       --work-tree=
+       --namespace=
+       --help
+       EOF
+       test_completion "git --"

There's a mistake here ^.

Yeah, good eyes!  ... ah, wait, it is your bug ;-)

Not my eyes, further tests =/

Do I need to resend the patch?

Not necessarily.  If you can eyeball what is queued on 'pu' and gave me an
Ack (or send a replacement with "You stupid, you amended it wrong!") that
should be sufficient.

Thanks.