Re: [PATCH 12/19] 1/2: Add Python support library for CVS remote helper
- From
Sverre Rabbelier <srabbelier@gmail.com>
- Date
- Oct 30, 2009, 12:27 UTC
- Message-ID
- <fabb9a1e0910300527w7033af71g8e65bcc69fde382b@mail.gmail.com>
- In-Reply-To
- <200910300933.35567.johan@herland.net>
Heya,
On Fri, Oct 30, 2009 at 01:33, Johan Herland <johan@herland.net> wrote:
> Why? Or: why that one, and not the others? Also, you might want to mention > your contribution in the commit message itself.
What others? And I thought my contributions were so minor it doesn't really matter that much :).
> It seems the two functions you add (notify() and warn()) have a different > indentation than the existing code (which uses 4 spaces). Please fix.
That's weird, will fix.
-- Cheers, Sverre Rabbelier