From: Egon W. <ego...@gm...> - 2008-12-08 13:19:33
|
All, On Thu, Dec 4, 2008 at 7:23 PM, Stefan Kuhn <ste...@eb...> wrote: > could some people please have a look at that merge? It's mainly new tests and > nothing spectacular. > it's not too great if things are not taken up for months. I have reviewed the patch, and Stefan updated the patch for my comments... But because it contains a few (minor) API cleanups, for it the get applied to cdk-1.2.x/ it needs another peer review by Miguel, Christoph or Rajarshi. I have 'grouped' the patch as (still) Needs Review which I describe in my blog: http://chem-bla-ics.blogspot.com/2008/12/peer-reviewed-cheminformatics-2-code.html Anyone can review, but the signoff has to be done by a senior developer... Egon -- ---- http://chem-bla-ics.blogspot.com/ |