From: Sergey Organov <sorganov@gmail.com> To: "SZEDER Gábor" <szeder.dev@gmail.com> Cc: "Ævar Arnfjörð Bjarmason" <avarab@gmail.com>, "Junio C Hamano" <gitster@pobox.com>, "Jeff King" <peff@peff.net>, "Philip Oakley" <philipoakley@iee.email>, "Elijah Newren" <newren@gmail.com>, "Felipe Contreras" <felipe.contreras@gmail.com>, git@vger.kernel.org Subject: Re: [PATCH 7/9] t9902: fix completion tests for log.d* to match log.diffMerges Date: Fri, 09 Apr 2021 02:07:22 +0300 [thread overview] Message-ID: <87h7kg5qad.fsf@osv.gnss.ru> (raw) In-Reply-To: <20210408221343.GC2947267@szeder.dev> ("SZEDER =?utf-8?Q?G?= =?utf-8?Q?=C3=A1bor=22's?= message of "Fri, 9 Apr 2021 00:13:43 +0200") SZEDER Gábor <szeder.dev@gmail.com> writes: > On Thu, Apr 08, 2021 at 11:26:38PM +0300, Sergey Organov wrote: [...] > >> It looks like something is wrong here, a bug or misfeature, or even two, >> and if it's fixed before these series, I won't need this in my series at >> all. Besides, that's yet another reason *not* to squash this change into >> an otherwise unrelated commit. > > The introduction of the new configuration variable, its documentation > and this test update should all go into a single patch. The whole > test suite must pass for every single commit. OK, fine, thanks for clarification! -- Sergey Organov
next prev parent reply other threads:[~2021-04-08 23:07 UTC|newest] Thread overview: 47+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-04-07 22:55 [PATCH 0/9] git log: configurable default format for merge diffs Sergey Organov 2021-04-07 22:56 ` [PATCH 1/9] diff-merges: introduce --diff-merges=def Sergey Organov 2021-04-08 11:48 ` Philip Oakley 2021-04-08 14:21 ` Sergey Organov 2021-04-08 17:27 ` Junio C Hamano 2021-04-08 17:38 ` Sergey Organov 2021-04-07 22:56 ` [PATCH 2/9] diff-merges: refactor set_diff_merges() Sergey Organov 2021-04-07 22:56 ` [PATCH 3/9] diff-merges: introduce log.diffMerges config variable Sergey Organov 2021-04-08 21:37 ` SZEDER Gábor 2021-04-08 21:51 ` SZEDER Gábor 2021-04-08 22:01 ` Junio C Hamano 2021-04-08 23:04 ` Sergey Organov 2021-04-07 22:56 ` [PATCH 4/9] diff-merges: adapt -m to enable default diff format Sergey Organov 2021-04-07 22:56 ` [PATCH 5/9] t4013: add test for --diff-merges=def Sergey Organov 2021-04-07 22:56 ` [PATCH 6/9] t4013: add tests for log.diffMerges config Sergey Organov 2021-04-07 23:06 ` Ævar Arnfjörð Bjarmason 2021-04-07 23:35 ` Junio C Hamano 2021-04-08 14:25 ` Sergey Organov 2021-04-07 22:56 ` [PATCH 7/9] t9902: fix completion tests for log.d* to match log.diffMerges Sergey Organov 2021-04-07 23:05 ` Ævar Arnfjörð Bjarmason 2021-04-08 14:41 ` Sergey Organov 2021-04-08 19:50 ` Ævar Arnfjörð Bjarmason 2021-04-08 20:26 ` Sergey Organov 2021-04-08 22:13 ` SZEDER Gábor 2021-04-08 23:07 ` Sergey Organov [this message] 2021-04-07 22:56 ` [PATCH 8/9] doc/diff-options: document new --diff-merges features Sergey Organov 2021-04-07 22:56 ` [PATCH 9/9] doc/config: document log.diffMerges Sergey Organov 2021-04-10 17:16 ` [PATCH v1 0/5] git log: configurable default format for merge diffs Sergey Organov 2021-04-10 17:16 ` [PATCH v1 1/5] diff-merges: introduce --diff-merges=default Sergey Organov 2021-04-10 17:16 ` [PATCH v1 2/5] diff-merges: refactor set_diff_merges() Sergey Organov 2021-04-10 17:16 ` [PATCH v1 3/5] diff-merges: adapt -m to enable default diff format Sergey Organov 2021-04-10 17:16 ` [PATCH v1 4/5] diff-merges: introduce log.diffMerges config variable Sergey Organov 2021-04-10 17:16 ` [PATCH v1 5/5] doc/diff-options: document new --diff-merges features Sergey Organov 2021-04-11 16:13 ` [PATCH v1 0/5] git log: configurable default format for merge diffs Junio C Hamano 2021-04-11 18:04 ` Sergey Organov 2021-04-11 19:02 ` Junio C Hamano 2021-04-11 20:38 ` Sergey Organov 2021-04-11 21:58 ` Sergey Organov 2021-04-13 11:41 ` [PATCH v2 " Sergey Organov 2021-04-13 11:41 ` [PATCH v2 1/5] diff-merges: introduce --diff-merges=on Sergey Organov 2021-04-13 23:18 ` Junio C Hamano 2021-04-13 11:41 ` [PATCH v2 2/5] diff-merges: refactor set_diff_merges() Sergey Organov 2021-04-13 11:41 ` [PATCH v2 3/5] diff-merges: adapt -m to enable default diff format Sergey Organov 2021-04-13 11:41 ` [PATCH v2 4/5] diff-merges: introduce log.diffMerges config variable Sergey Organov 2021-04-15 20:21 ` Junio C Hamano 2021-04-16 8:30 ` Sergey Organov 2021-04-13 11:41 ` [PATCH v2 5/5] doc/diff-options: document new --diff-merges features Sergey Organov
Reply instructions: You may reply publicly 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=87h7kg5qad.fsf@osv.gnss.ru \ --to=sorganov@gmail.com \ --cc=avarab@gmail.com \ --cc=felipe.contreras@gmail.com \ --cc=git@vger.kernel.org \ --cc=gitster@pobox.com \ --cc=newren@gmail.com \ --cc=peff@peff.net \ --cc=philipoakley@iee.email \ --cc=szeder.dev@gmail.com \ --subject='Re: [PATCH 7/9] t9902: fix completion tests for log.d* to match log.diffMerges' \ /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
Code repositories for project(s) associated with this inbox: https://80x24.org/mirrors/git.git This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).