#6529: [with patch, positive review, with second patch, needs review] adding
doctests to arith.py
------------------------------+---------------------------------------------
Reporter: mhampton | Owner: somebody
Type: enhancement | Status: new
Priority: major | Milestone: sage-4.1.1
Component: basic arithmetic | Keywords:
Reviewer: David Loeffler | Author: Marshall Hampton, David Loeffler
Merged: |
------------------------------+---------------------------------------------
Changes (by newvalueoldvalue):
* reviewer: wstein, craigcitro => David Loeffler
* author: => Marshall Hampton, David Loeffler
Comment:
Good work: patch applies fine, and all doctests pass. But I noticed that
several docstrings are mis-formatted, including the one for the new
four_squares function. Then I had an attack of enthusiasm and decided to
clean all that up. Hence the second patch above.
I'm happy with mhampton's changes; so if mhampton (or anyone someone else)
could take a quick look at the second patch, then we can call this a
positive review.
David
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6529#comment:2>
Sage <http://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
-~----------~----~----~----~------~----~------~--~---
