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: sage-6.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:

Status badges

Description (last modified by chapoton)

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 chapoton

  • Description modified (diff)
  • Keywords doctest added
  • Report Upstream set to N/A

comment:2 Changed 7 years ago by chapoton

  • Authors set to Frédéric Chapoton
  • Branch set to u/chapoton/3912
  • Commit set to 6767e49fed16e13fb6523e2c896594f01cde6ee7
  • Description modified (diff)
  • Keywords latex added
  • Milestone changed from sage-wishlist to sage-6.3
  • Status changed from new to needs_review

New commits:

6767e49trac #3912 tentative of using lstlisting in latex doctests

comment:3 Changed 7 years ago by chapoton

comment:4 Changed 7 years ago by chapoton

  • Cc mvngu added

comment:5 Changed 7 years ago by aapitzsch

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 git

  • Commit changed from 6767e49fed16e13fb6523e2c896594f01cde6ee7 to 82abb804bf53b61de3dcc05ec1986d05f4f97f7d

Branch pushed to git repo; I updated commit sha1. New commits:

82abb80trac #3912 reviewer's comments

comment:7 Changed 7 years ago by aapitzsch

  • Reviewers set to André Apitzsch
  • Status changed from needs_review to positive_review

comment:8 Changed 7 years ago by vbraun

  • 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.