[darcs-users] darcs patch: Resolve issue1093: warn about ugly patch... (and 1 more)
benjamin.franksen at bessy.de
Mon Feb 2 22:46:24 UTC 2009
Eric Kow wrote:
> On Mon, Feb 02, 2009 at 20:31:39 +1100, Trent W. Buck wrote:
>> Under what circumstances is it desirable for a patch name to *not* be
>> a sentence?
> Not all projects are prestigious and sometimes people just want to throw
> together a quick little repository. I've also flip-flopped between
> putting punctuation and not (seeing it as superfluous), although now
> that I hear somebody else saying that it's good to have, I'll be a good
> boy from now on. (That said, it's not clear if we always want a
> sentence, maybe sometimes a noun phrase is more direct?)
> I think this patch is a bit too helpful. It's really just a gut
> feeling, and it could be wrong. My personal approval vote is:
> YES - warn on patches starting with '-'
> YES - warn on "strange" patch names http://bugs.darcs.net/issue1000
> MAYBE - warn on overly long patch names (as this could cause some
> concrete practical problems for other folks, wrt wrapping)
> NO - warn on patch names < 10
> NO - warn on punctuation
> I don't want this to turn into bikeshed either. Maybe we could hear
> from three more users?
What is wrong about "fixed issue #12345" ?
In fact, I /never/ use punctuation at the end of a patch name and always
start lower case. This is clearly a matter of convention and the tool
should be completely agnostic w.r.t. such conventions. Anything else is
intrusive and thus annoying.
More information about the darcs-users