From: Jeff King <peff@peff.net>
Date: Mon, 30 Jul 2012 16:39:07 GMT
Subject: Re: [PATCH] Improve tests for detached worktree in git-submodule
Message-ID: <20120730163907.GA18109@sigill.intra.peff.net>
In-Reply-To: <1343664610-479-1-git-send-email-dangra@gmail.com>

On Mon, Jul 30, 2012 at 01:10:10PM -0300, Daniel Graña wrote:

> Subject: Re: [PATCH] Improve tests for detached worktree in git-submodule
> 
> Signed-off-by: Daniel Graña <dangra@gmail.com>

The space between the subject and your S-o-b is an excellent place to
explain the rationale for your commit.

How are we improving them? What cases or classes of failure does this
catch that the original did not?  It may be because I have not been
following this topic closely, but reading the patch, I am not sure what
the purpose is. Please make life easier for reviewers by telling us what
to expect and why before we even get to the patch.

-Peff