From: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Date: Sat, 18 Mar 2017 19:52:11 GMT
Subject: Re: [PATCH 8/8] tag: Change --point-at to default to HEAD
Message-ID: <CACBZZX5y7ZntOO1RsX74AHoYDJb6pUrjFDbNcamXVJ4Yieeviw@mail.gmail.com>
In-Reply-To: <xmqqinn62z74.fsf@gitster.mtv.corp.google.com>

On Sat, Mar 18, 2017 at 7:54 PM, Junio C Hamano <gitster@pobox.com> wrote:
> Ævar Arnfjörð Bjarmason  <avarab@gmail.com> writes:
>
>> Change the --points-at option to default to HEAD for consistency with
>> its siblings --contains, --merged etc. which default to HEAD. This
>> changes behavior added in commit ae7706b9ac (tag: add --points-at list
>> option, 2012-02-08).
>
> Makes a lot of sense to me.
>
>> +test_expect_success '--points-at is a synonym for --points-at HEAD' '
>> +     git tag --points-at >actual &&
>
> Even if "expect" is the same one established earlier, it is easier
> to read and understand individual tests if you explicitly said what
> this one expects.

Makes sense. Queued that change in my WIP v2.