mercurial/crew: 9 outgoing changesets
Vadim Gelfer
vadim.gelfer at gmail.com
Tue Jun 27 16:38:09 UTC 2006
On 6/27/06, Alexis S. L. Carvalho <alexis at cecm.usp.br> wrote:
> This is a nice thing to have. A few comments:
>
> - the Subject of the message was ignored when I tried to import a
> message with an inline patch - I haven't tried it with attachments
thanks you for seeing problems. this is fixed now.
> - it'd be nice to clean up the subject a bit - specifically remove
> leading spaces, "Re:", "[listname]" and maybe "[PATCH 17/42]", being
> careful with stuff like "Re: [listname] Re: [PATCH] patch description"
if you have idea for kind of safe regexp, i will use.
> - the docs mention the "Sender and subject line of email" - I think it'd
> be better to mention something like the "The From and Subject headers
> of the email": "Sender" is also a valid header that is added by e.g.
> mailing lists.
ok thanks you.
More information about the Mercurial
mailing list