Michael G Schwern <schwern@xxxxxxxxx> wrote: Hi Michael, thanks for taking your time to help with this. I agree with everything Jonathan said (and thank him for taking the time to point you in the right direction). I too (very strongly) prefer email for code review. I doubt I would've ever gotten involved if git were run differently. I'm actually disappointed the mailing list culture that built git hasn't rubbed off to other projects that adopt git. > +++ b/t/Git-SVN/00compile.t > +use Test::More tests => 2; I prefer not declaring test counts and using done_testing() instead. done_testing() is favorable to me in at least 2 ways: * done_testing() closely matches the behavior of the existing sh-based test suite in git (which calls test_done) * maintaining test counts leads to unnecessary merge conflicts Skipping the tests on old versions of Test::More (< 0.88) is acceptable to me (especially since integration tests provide the real coverage already). > +++ b/t/Git-SVN/Utils/can_compress.t > +use Test::More 'no_plan'; no_plan is the worst way to use Test::More, I think. -- 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