#17534: The reviewer's checklist
-------------------------+-------------------------------------------------
Reporter: | Owner:
ncohen | Status: needs_review
Type: | Milestone: sage-6.5
enhancement | Resolution:
Priority: major | Merged in:
Component: | Reviewers: Jeroen Demeyer, Karl-Dieter
documentation | Crisman, Martin Rubey
Keywords: | Work issues:
Authors: | Commit:
Nathann Cohen | 4ff64c1ef8d6d03622c01db321ef76807b6adfef
Report Upstream: N/A | Stopgaps:
Branch: |
public/17534 |
Dependencies: |
-------------------------+-------------------------------------------------
Comment (by kcrisman):
> Well, I fixed your comments in a new commit, in the hope that you would
give this ticket a positive review during your next 10-minutes break :-P
What time zone are you in right now? I wasn't expecting any response
until I signed off...
> Hmmmmm... I would say that a clear page on the different types of
doctests, what they check and how they do it would also be of some help to
reviewers. Knowing what people usually test in doctests would tell them
what they should pay attention to.
Sure, not on this ticket, of course.
Okay, I'll bite and push something.
By the way, on a first run-through I get
{{{
WARNING: duplicate citation WSblog, other instance in
/Users/karl.crisman/Downloads/sage/src/doc/en/developer/trac.rst
}}}
but I think that only happens when there is already an existing thing, not
necessarily when this would actually get done... what about when one
upgrades, though?
--
Ticket URL: <http://trac.sagemath.org/ticket/17534#comment:19>
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.
