#17549: Rephrase the 'doctest flags' section of the developer's manual
-------------------------+-------------------------------------------------
Reporter: | Owner:
ncohen | Status: needs_review
Type: | Milestone: sage-6.5
enhancement | Resolution:
Priority: major | Merged in:
Component: | Reviewers: Jeroen Demeyer
documentation | Work issues:
Keywords: | Commit:
Authors: | 0ebf12ac4f21f78ebe2078b910624c1e97e3512d
Nathann Cohen | Stopgaps:
Report Upstream: N/A |
Branch: |
u/ncohen/17549 |
Dependencies: |
-------------------------+-------------------------------------------------
Comment (by ncohen):
Hello guys !
> What about the --optional=sage,keyword?
It is already documented in the chapter (that this patch renames to)
"Running the doctests". The section about the 'optional' flag begins with
a link toward that page
http://www.sagemath.org/doc/developer/doctesting.html#run-optional-tests
I think that it is sufficient for the moment, especially since we all seem
to agree that this behaviour should be changed so that 'sage' is always
included in `--optional=<a list>`.
> {{{A line flagged with ``optional - keyword``}}}, it is not tested:
remove {{{", it"}}}
Done.
> I really think that it should be involves since "representation" is
singular.
Done.
> Also, maybe we should emphasize that the "32-bit" and "64-bit" flags go
on the output lines.
Done
From the look of your comments I have no idea if you consider that this
ticket can be switched to `positive_review` yet, but I would really
appreciate it if you could make all your comments at once, next time. I
have already pushed 6 different review commits here, and that is much more
work than implementing at once all of the reviewer's comments.
Thanks,
Nathann
--
Ticket URL: <http://trac.sagemath.org/ticket/17549#comment:43>
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.
