[Sugar-devel] [PATCH] Autocomplete functionality for the address does not work #2406
Simon Schampijer
simon at schampijer.de
Wed Oct 6 10:40:55 EDT 2010
On 10/06/2010 04:32 PM, Lucian Branescu wrote:
> On 6 October 2010 15:26, Sascha Silbe
> <sascha-ml-reply-to-2010-3 at silbe.org> wrote:
>> Excerpts from Simon Schampijer's message of Wed Oct 06 15:31:07 +0200 2010:
>>
>>> Hmm, Lucian was to quick.
>>
>> Hehe. No problem; I don't think this particular patch is going to
>> matter when hunting down some issue and there's enough information
>> in the description, even if just through an indirection (bug tracker).
>> I just want to make sure we move in the right general direction. Good
>> commit messages are getting more important as the number of contributors
>> increases since no single person will have an overview of all the code
>> (changes) anymore.
>> This is why I pester new contributors and core developers alike with my
>> complaints about commit messages.
>
> Yes, it's a good thing you do keep pestering people. And I should know
> better, too.
+1 That is what reviews are for, to keep the quality high.
Please keep on.
Regards,
Simon
More information about the Sugar-devel
mailing list