Great. Well then unless someone objects let's henceforth gently remind
people of this practice until it becomes habit :)

Dan

Στις 29 Απριλίου 2011 7:01 μ.μ., ο χρήστης Yuri Z <vega...@gmail.com> έγραψε:
> +1
> I ll start doing it from next commit.
>
> 2011/4/29 Lennard de Rijk <ljvder...@gmail.com>
>
>> Hi Daniel,
>>
>> On Fri, Apr 29, 2011 at 09:43, Daniel Danilatos <danila...@google.com
>> >wrote:
>>
>> > I think it would be a good practice if everyone puts a link to the
>> > code review in the commit message. This makes it very easy to see the
>> > discussions/justifications that happened on a particular change.
>> >
>> > Thoughts?
>> >
>>
>> I totally agree with this. During my time on the project I've always tried
>> to indicate the number of the review in Rietveld and who reviewed it. It
>> sounds like a very nice idea if this practice could be continued when we
>> migrated to Apache.
>>
>> Greetings,
>> Lennard
>>
>

Reply via email to