#6354 closed enhancement (fixed)
Advertise and improve sage -fixdoctest
Reported by: | nthiery | Owned by: | nthiery |
---|---|---|---|
Priority: | major | Milestone: | sage-4.3 |
Component: | doctest coverage | Keywords: | fix doctests |
Cc: | sage-combinat, mhansen, rlm | Merged in: | sage-4.3.alpha0 |
Authors: | Nicolas M. Thiéry | Reviewers: | Mike Hansen |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
After this patch, sage -fixdoctest handles multiline doctests, and use the line number info of sage -t to be more robust (handles multiple doctests with the same expected output in the same file).
By the way, sage -advanced advertises sage -fixdoctest.
Attachments (1)
Change History (9)
Changed 14 years ago by
Attachment: | sage-fixdoctests-6354-nt.patch added |
---|
comment:1 Changed 14 years ago by
Authors: | nthiery → Nicolas Thiery |
---|---|
Reviewers: | → Mike Hansen |
Summary: | [with patch, needs review] Advertise and improve sage -fixdoctest → [with patch, positive review] Advertise and improve sage -fixdoctest |
comment:2 follow-up: 3 Changed 14 years ago by
Merged in: | → sage-4.1.alpha0 |
---|---|
Resolution: | → fixed |
Status: | new → closed |
comment:3 Changed 13 years ago by
Authors: | Nicolas Thiery → Nicolas M. Thiéry |
---|---|
Cc: | rlm added |
Resolution: | fixed |
Status: | closed → new |
Replying to rlm:
Err, I don't see it in sage-4.2.1; was it somehow lost?
comment:5 Changed 13 years ago by
Milestone: | sage-4.1 → sage-4.3 |
---|---|
Status: | new → needs_review |
Replying to rlm:
Sorry, it must have gotten lost during merging....
No worry :-) I set it back to positive review so that it get merged in 4.3.
comment:6 Changed 13 years ago by
Status: | needs_review → positive_review |
---|
comment:7 Changed 13 years ago by
Merged in: | sage-4.1.alpha0 → sage-4.3.alpha0 |
---|---|
Resolution: | → fixed |
Status: | positive_review → closed |
comment:8 Changed 13 years ago by
Report Upstream: | → N/A |
---|---|
Summary: | [with patch, positive review] Advertise and improve sage -fixdoctest → Advertise and improve sage -fixdoctest |
Note: See
TracTickets for help on using
tickets.
Looks good to me.