On Thu, Apr 30, 2015 at 10:03 AM, Quentin Neill <quentin.neill@xxxxxxxxx> wrote: > I'm thinking of a test file that will test all the git blame options, > but for this > patch it will only test the new showEmail config. I read t/README and > tentatively named the new test file "t/8009-blame-config.sh". I'm not sure that these new proposed tests warrant a new test script. Moreover, the tests presumably will be testing combinations of configuration options and command-line switches, so having "config" in the script name doesn't seem quite correct. t8002-blame.sh already contains a test for --show-email, so it may be logical to add the new tests there, alongside the existing one. On the other hand, if you do add a new test script, then perhaps the existing --show-email test in t8002-blame.sh should be moved to the new script. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html