#4925: [with patch, needs work] convert sage.rings.* docstrings to Sphinx
---------------------------+------------------------------------------------
 Reporter:  mhansen        |        Owner:  tba     
     Type:  enhancement    |       Status:  new     
 Priority:  major          |    Milestone:  sage-3.4
Component:  documentation  |   Resolution:          
 Keywords:                 |  
---------------------------+------------------------------------------------
Comment (by cwitty):

 I've attached a patch that fixes the problems I found in the remaining 78%
 of the file (my patch does not fix problems that I reported in the above
 comment).

 I now suspect that you removed email addresses from AUTHOR blocks (in
 polynomial/multi_polynomial_ideal.py and real_mpfr.pyx) deliberately.  If
 so,

 1) I would have preferred that this be a separate change; having
 substantive changes mixed in with the huge LaTeX->ReST patch makes it more
 difficult to review; and

 2) you missed some (for example, pbori.pyx still has email addresses).

 Once the issues from the above comment are fixed, and we decide what to do
 about the email addresses, this will be a positive review for the original
 patch.  Probably somebody should review my reviewer patch, though, since
 it's so big.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4925#comment:4>
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