From: Junio C Hamano <gitster@pobox.com> To: Ævar Arnfjörð Bjarmason <avarab@gmail.com> Cc: git@vger.kernel.org, Lars Hjemli <hjemli@gmail.com>, Jeff King <peff@peff.net>, Christian Couder <christian.couder@gmail.com>, Carlos Rica <jasampler@gmail.com>, Samuel Tardieu <sam@rfc1149.net>, Tom Grennan <tmgrennan@gmail.com> Subject: Re: [PATCH 8/8] tag: Change --point-at to default to HEAD Date: Sat, 18 Mar 2017 11:54:07 -0700 Message-ID: <xmqqinn62z74.fsf@gitster.mtv.corp.google.com> (raw) In-Reply-To: <20170318103256.27141-9-avarab@gmail.com> Æ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. Thanks.
next prev parent reply index Thread overview: 37+ messages / expand[flat|nested] mbox.gz Atom feed top 2017-03-18 10:32 [PATCH 0/8] Various changes to the "tag" command Ævar Arnfjörð Bjarmason 2017-03-18 10:32 ` [PATCH 1/8] tag: Remove a TODO item from the test suite Ævar Arnfjörð Bjarmason 2017-03-18 18:14 ` Junio C Hamano 2017-03-18 18:42 ` [PATCH 0/2] doc/SubmittingPatches: A couple of minor improvements Ævar Arnfjörð Bjarmason 2017-03-18 18:42 ` [PATCH 1/2] doc/SubmittingPatches: clarify the casing convention for "area: change..." Ævar Arnfjörð Bjarmason 2017-03-18 19:04 ` Junio C Hamano 2017-03-18 19:16 ` Ævar Arnfjörð Bjarmason 2017-03-18 20:07 ` Junio C Hamano 2017-03-18 18:42 ` [PATCH 2/2] doc/SubmittingPatches: show how to get a CLI commit summary Ævar Arnfjörð Bjarmason 2017-03-18 19:07 ` Junio C Hamano 2017-03-19 0:48 ` [PATCH 1/8] tag: Remove a TODO item from the test suite Jakub Narębski 2017-03-19 6:43 ` Ævar Arnfjörð Bjarmason 2017-03-18 10:32 ` [PATCH 2/8] tag: Refactor the options handling code to be less bizarro Ævar Arnfjörð Bjarmason 2017-03-18 18:35 ` Junio C Hamano 2017-03-18 19:13 ` Ævar Arnfjörð Bjarmason 2017-03-18 19:27 ` Junio C Hamano 2017-03-18 20:00 ` Ævar Arnfjörð Bjarmason 2017-03-18 10:32 ` [PATCH 3/8] tag: Change misleading --list <pattern> documentation Ævar Arnfjörð Bjarmason 2017-03-18 18:43 ` Junio C Hamano 2017-03-18 19:49 ` Ævar Arnfjörð Bjarmason 2017-03-20 3:44 ` Jeff King 2017-03-20 15:55 ` Junio C Hamano 2017-03-20 17:07 ` Junio C Hamano 2017-03-20 16:09 ` Ævar Arnfjörð Bjarmason 2017-03-20 16:11 ` Jeff King 2017-03-18 10:32 ` [PATCH 4/8] tag: Implicitly supply --list given another list-like option Ævar Arnfjörð Bjarmason 2017-03-20 3:55 ` Jeff King 2017-03-20 9:16 ` Ævar Arnfjörð Bjarmason 2017-03-18 10:32 ` [PATCH 5/8] tag: Implicitly supply --list given the -n option Ævar Arnfjörð Bjarmason 2017-03-20 4:02 ` Jeff King 2017-03-18 10:32 ` [PATCH 6/8] ref-filter: Add --no-contains option to tag/branch/for-each-ref Ævar Arnfjörð Bjarmason 2017-03-20 4:25 ` Jeff King 2017-03-20 9:32 ` Ævar Arnfjörð Bjarmason 2017-03-20 19:52 ` Jeff King 2017-03-20 20:04 ` Ævar Arnfjörð Bjarmason 2017-03-18 10:32 ` [PATCH 8/8] tag: Change --point-at to default to HEAD Ævar Arnfjörð Bjarmason 2017-03-18 18:54 ` Junio C Hamano [this message] 2017-03-18 19:52 ` Ævar Arnfjörð Bjarmason 2017-03-20 4:26 ` [PATCH 0/8] Various changes to the "tag" command Jeff King 2017-03-20 15:57 ` Junio C Hamano 2017-03-21 15:51 [PATCH v2 2/2] doc/SubmittingPatches: show how to get a CLI commit summary SZEDER Gábor 2017-03-21 17:58 ` Junio C Hamano 2017-03-21 18:47 ` Ævar Arnfjörð Bjarmason 2017-03-21 20:01 ` SZEDER Gábor 2017-03-21 20:13 ` Junio C Hamano
Reply instructions: You may reply publically to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style List information: http://vger.kernel.org/majordomo-info.html * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=xmqqinn62z74.fsf@gitster.mtv.corp.google.com \ --to=gitster@pobox.com \ --cc=avarab@gmail.com \ --cc=christian.couder@gmail.com \ --cc=git@vger.kernel.org \ --cc=hjemli@gmail.com \ --cc=jasampler@gmail.com \ --cc=peff@peff.net \ --cc=sam@rfc1149.net \ --cc=tmgrennan@gmail.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
git@vger.kernel.org mailing list mirror (one of many) Archives are clonable: git clone --mirror https://public-inbox.org/git git clone --mirror http://ou63pmih66umazou.onion/git git clone --mirror http://czquwvybam4bgbro.onion/git git clone --mirror http://hjrcffqmbrq6wope.onion/git Newsgroups are available over NNTP: nntp://news.public-inbox.org/inbox.comp.version-control.git nntp://ou63pmih66umazou.onion/inbox.comp.version-control.git nntp://czquwvybam4bgbro.onion/inbox.comp.version-control.git nntp://hjrcffqmbrq6wope.onion/inbox.comp.version-control.git nntp://news.gmane.org/gmane.comp.version-control.git note: .onion URLs require Tor: https://www.torproject.org/ or Tor2web: https://www.tor2web.org/ AGPL code for this site: git clone https://public-inbox.org/ public-inbox