Re: [infinispan-dev] optimistic locking - the time has come
Agreed, even better if we could use the commit description so it stays
with the code, and you can describe a step at a time.
We usually use a single-liner description, but for the more complex
patches we should add a paragraph. I did it occasionally, not sure
anybody noticed :)
Sanne
On 28 September 2011 11:27, Manik Surtani <manik(a)jboss.org> wrote:
On 28 Sep 2011, at 10:05, Galder Zamarreño wrote:
If you're gonna submit a big pull req, please use the description to
*summarise the changes* in order to help reviewers. This helps focus the
attention of the reviewer to the pieces of code that are most relevant.
+1!
--
Manik Surtani
manik(a)jboss.org
twitter.com/maniksurtani
Lead, Infinispan
http://www.infinispan.org
_______________________________________________
infinispan-dev mailing list
infinispan-dev(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/infinispan-dev