On Tue, Feb 08, 2011 at 06:24:46AM -0800, Niles wrote:
> So that my work and theirs can be fruitful, I have a suggestion for
> distributing the review effort. I know there are several other large
> patches languishing in need of review, and maybe the same idea can
> help them too.
>
> The suggestion is that the patch author provide a list of review
> items, which can be positively reviewed independently. One of these
> should be "The items on this list constitute a complete review", and
> other obvious items are "All code is documented and doctested
> thoroughly; documentation builds without error or warning" and "Sage
> passes all doctests with this patch" (the patch buildbot can probably
> take care of this one). Other items on the list may be more patch-
> specific. For example, here is the list of review items I've proposed
> for #1956 (implement multivariate truncated power series arithmetic):
+1.
Just for the record, I had used a similar procedure for the category
patch (and its many dependencies), using a wiki page:
http://trac.sagemath.org/sage_trac/wiki/CategoriesRoadMap
Cheers,
Nicolas
--
Nicolas M. Thiéry "Isil" <nthi...@users.sf.net>
http://Nicolas.Thiery.name/
--
To post to this group, send an email to sage-devel@googlegroups.com
To unsubscribe from this group, send an email to
sage-devel+unsubscr...@googlegroups.com
For more options, visit this group at http://groups.google.com/group/sage-devel
URL: http://www.sagemath.org
