#4372: [with patch, needs review] Repair to totallyreal_dsage
---------------------------+------------------------------------------------
 Reporter:  jvoight        |        Owner:  was     
     Type:  defect         |       Status:  new     
 Priority:  minor          |    Milestone:  sage-3.2
Component:  number theory  |   Resolution:          
 Keywords:                 |  
---------------------------+------------------------------------------------
Changes (by mabshoff):

  * summary:  [with patch] Repair to totallyreal_dsage => [with patch,
              needs review] Repair to totallyreal_dsage

Comment:

 > totallyreal_dsage is still at a nodoctest status, so shouldn't require
 any (meaningful) review if the patch works (it's against 3.1.4).

 Any patch should be reviewed in detail regardless of nodoctest status or
 not. Is there a specific reason we shouldn't remove the nodoctest on that
 file now?

 Cheers,

 Michael

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4372#comment:1>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of 
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
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