#12585: Bring matrix/matrix0.pyx to 100% coverage
------------------------------+---------------------------------------------
Reporter: hthomas | Owner: mvngu
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-5.0
Component: doctest | Keywords:
Work_issues: | Upstream: N/A
Reviewer: David Loeffler | Author: Hugh Thomas
Merged: | Dependencies:
------------------------------+---------------------------------------------
Changes (by davidloeffler):
* reviewer: => David Loeffler
Comment:
The reason sage -coverage was complaining is because of the comment placed
*before* the docstring. I.e. it's not clever enough to parse
{{{
#!python
def some_function(args):
# comment
""" docstring """
function_body
}}}
I'm about to upload a tiny reviewer patch which moves the comment in
{{{_lmul_}}} to after the docstring, and thus gets coverage up to 100%.
I've also tagged a few doctests as {{{# indirect doctest }}} to stop it
complaining about the function name not appearing. Hugh: I'm happy with
your patch, so if you're happy with my patch please set this to "positive
review".
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12585#comment:4>
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.
