On Thu, Aug 10, 2017 at 11:28:52AM -0700, Stefan Beller wrote: > > +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. Yeah, I actually considered that but decided to keep style with the rest of the script. I agree the whole thing could possibly switch to test_config, but I suspect there may be some fallouts (the style of the rest of the script seems to assume some continuity between the tests). -Peff
