From: Junio C Hamano <gitster@pobox.com>
Date: Fri, 16 Oct 2020 16:37:50 GMT
Subject: Re: [PATCH v2 1/2] sequencer: fix gpg option passed to merge subcommand
Message-ID: <xmqqy2k6cejl.fsf@gitster.c.googlers.com>
In-Reply-To: <31ce457b-e71c-0ca0-e5be-a9aebb9cf785@gmail.com>

Phillip Wood <phillip.wood123@gmail.com> writes:

> ... Reviewers that only read the patch without loading up the
> test file in their editor have no indication that the test should be
> clearing the config variable.

It is not a review if the code being updated is not checked for
sanity in its own context, is it?