Opened 13 years ago
Closed 7 years ago
#3912 closed enhancement (fixed)
sage t foo.tex should also test listings blocks not only verbatim
Reported by:  malb  Owned by:  tba 

Priority:  minor  Milestone:  sage6.3 
Component:  documentation  Keywords:  doctest, latex 
Cc:  mvngu  Merged in:  
Authors:  Frédéric Chapoton  Reviewers:  André Apitzsch 
Report Upstream:  N/A  Work issues:  
Branch:  82abb80 (Commits, GitHub, GitLab)  Commit:  82abb804bf53b61de3dcc05ec1986d05f4f97f7d 
Dependencies:  Stopgaps: 
Description (last modified by )
sage t can check stuff in
\begin{verbatim} sage: 2 + 3 5 \end{verbatim}
I propose to also support
\begin{lstlisting} sage: 2 + 3 5 \end{lstlisting}
which looks so much nicer.
Change History (8)
comment:1 Changed 7 years ago by
 Description modified (diff)
 Keywords doctest added
 Report Upstream set to N/A
comment:2 Changed 7 years ago by
 Branch set to u/chapoton/3912
 Commit set to 6767e49fed16e13fb6523e2c896594f01cde6ee7
 Description modified (diff)
 Keywords latex added
 Milestone changed from sagewishlist to sage6.3
 Status changed from new to needs_review
comment:3 Changed 7 years ago by
See http://mvngu.wordpress.com/2010/06/27/typesettingsagecodelistingsinlatex/ for nice parameters for the package listings
.
comment:4 Changed 7 years ago by
 Cc mvngu added
comment:5 Changed 7 years ago by
Line 153 contains a left parenthesis which is never closed.
Could you also unify Line 1142 and Line 1219.
comment:6 Changed 7 years ago by
 Commit changed from 6767e49fed16e13fb6523e2c896594f01cde6ee7 to 82abb804bf53b61de3dcc05ec1986d05f4f97f7d
Branch pushed to git repo; I updated commit sha1. New commits:
82abb80  trac #3912 reviewer's comments

comment:7 Changed 7 years ago by
 Reviewers set to André Apitzsch
 Status changed from needs_review to positive_review
comment:8 Changed 7 years ago by
 Branch changed from u/chapoton/3912 to 82abb804bf53b61de3dcc05ec1986d05f4f97f7d
 Resolution set to fixed
 Status changed from positive_review to closed
Note: See
TracTickets for help on using
tickets.
New commits:
trac #3912 tentative of using lstlisting in latex doctests