Re: Proposal regarding code reviews and submitting
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
Re: Proposal regarding code reviews and submitting
Hi Michael, I assume by the Jira issue number you mean the Jira issue number, if the patch addresses one We won't have to file a bug for every patch we submit, right? Dan Στις 30 Απριλίου 2011 1:10 π.μ., ο χρήστης Michael MacFadden michael.macfad...@gmail.com έγραψε: On a related note, we should also get in the habit of putting the Jira Issue number in as well. When we move to SVN, Jira will automatically pick up the reference in the commit message and show the source files that were changed in relation to the bug / enhancement. On Apr 29, 2011, at 7:46 AM, Daniel Danilatos wrote: 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
Re: Proposal regarding code reviews and submitting
Dan, Correct! If you are actively working on an issue in the system, then you should put in the issue number, otherwise not. ~Michael On Apr 29, 2011, at 8:30 AM, Daniel Danilatos wrote: Hi Michael, I assume by the Jira issue number you mean the Jira issue number, if the patch addresses one We won't have to file a bug for every patch we submit, right? Dan Στις 30 Απριλίου 2011 1:10 π.μ., ο χρήστης Michael MacFadden michael.macfad...@gmail.com έγραψε: On a related note, we should also get in the habit of putting the Jira Issue number in as well. When we move to SVN, Jira will automatically pick up the reference in the commit message and show the source files that were changed in relation to the bug / enhancement. On Apr 29, 2011, at 7:46 AM, Daniel Danilatos wrote: 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