Michael G Schwern <schw...@pobox.com> 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
> +++ 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 majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html