Re: [PATCH v3 2/5] interpret-trailers: add an option to show only the trailers
- Date: Thu, 10 Aug 2017 11:28:52 -0700
- From: Stefan Beller <sbeller@xxxxxxxxxx>
- Subject: Re: [PATCH v3 2/5] interpret-trailers: add an option to show only the trailers
On Thu, Aug 10, 2017 at 11:04 AM, Jeff King <peff@xxxxxxxx> wrote:
> In theory it's easy for any reader who wants to parse
> trailers to do so. But there are a lot of subtle corner
> cases around what counts as a trailer, when the trailer
> block begins and ends, etc. Since interpret-trailers already
> has our parsing logic, let's let callers ask it to just
> output the trailers.
> They still have to parse the "key: value" lines, but at
> least they can ignore all of the other corner cases.
> Signed-off-by: Jeff King <peff@xxxxxxxx>
Sorry for a sloppy review last round, upon reviewing
I found another nit.
> +test_expect_success 'only trailers' '
> + git config trailer.sign.command "echo config-value" &&
You may want to use 'test_config' here, which keeps the config
only for one test. The subsequent tests seem to overwrite the
config, so this is not wrong, just not the best style.