#19503: strip TESTS blocks in introspection
-------------------------------------+-------------------------------------
Reporter: jhpalmieri | Owner:
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-6.10
Component: documentation | Resolution:
Keywords: | Merged in:
Authors: John Palmieri | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/jhpalmieri/TESTS | 95f8c726ec2344d9a1e37da64a5528bc5f7acc84
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by jhpalmieri):
Replying to [comment:15 jdemeyer]:
> I mean: consider the environment variable as an implementation detail,
not as part of the public interface.
So you are suggesting deleting the addition to the installation guide,
right? We could also add a target to the makefile that does this, by the
way.
> Can you explain the change in src/sage/finance/stock.py?
The lines after the original test,
{{{
Classes and methods
-------------------
}}}
are not detected by the regular expression for the end of the test, so
they would be hidden upon introspection, so I moved the test from the top-
level to the class being tested. There is a similar change in
src/sage/matroids/basis_exchange_matroid.pyx. Note that those lines
provide a header that only really make sense when viewing the full
reference manual, so this particular change helps with the reference
manual and preserves the old stupid behavior if you type
`sage.finance.stock?`. Another option would be to delete
{{{
Classes and methods
-------------------
}}}
--
Ticket URL: <http://trac.sagemath.org/ticket/19503#comment:17>
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 unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
