#12662: Improve doctest coverage for sage.rings.qqbar
----------------------------------+-----------------------------------------
       Reporter:  davidloeffler   |         Owner:  mvngu          
           Type:  defect          |        Status:  positive_review
       Priority:  major           |     Milestone:  sage-5.0       
      Component:  doctest         |    Resolution:                 
       Keywords:                  |   Work issues:                 
Report Upstream:  N/A             |     Reviewers:  Jim Stark      
        Authors:  David Loeffler  |     Merged in:                 
   Dependencies:                  |      Stopgaps:                 
----------------------------------+-----------------------------------------
Changes (by JStarx):

  * status:  needs_review => positive_review
  * reviewer:  => Jim Stark


Comment:

 Ok, it all looks good.  The only minor tweek I would make is to delete the
 commented stuff on lines 7594 - 7641.  Not sure why that's there or why we
 wouldn't delete it.

 Anyway, I'm gonna set it as a positive review.  If you agree that that
 commented stuff can be deleted then feel free to keep the ticket marked as
 a positive review after deleting it.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12662#comment:10>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en.

Reply via email to