Opened 4 years ago

Closed 4 years ago

#22465 closed enhancement (fixed)

Doctest continuation marker / several files / 3

Reported by: jmantysalo Owned by:
Priority: minor Milestone: sage-7.6
Component: documentation Keywords: days85
Cc: tscrim Merged in:
Authors: Jori Mäntysalo Reviewers: Julian Rüth
Report Upstream: N/A Work issues:
Branch: 4e7d27a (Commits, GitHub, GitLab) Commit: 4e7d27a4cfcf1b0a8d591fbd46c98e66ec8bf2ce
Dependencies: Stopgaps:

Status badges


This is the last patch about doctest continuation markers at src/sage. (There are some left at src/doc.)

In few cases this corrects a bug where a doctest does not really check for right output.

Change History (6)

comment:1 Changed 4 years ago by jmantysalo

  • Branch set to u/jmantysalo/doctest-continuation-last

comment:2 Changed 4 years ago by jmantysalo

  • Cc tscrim added
  • Commit set to 4db20c0eb91d44268ace513ee2ba4071d80fb5b1
  • Status changed from new to needs_review

Is it possible to forbid old-style continuation markers? That would prevent some doctest errors.

New commits:

4db20c0Last doctest continuation marker change on src/sage.

comment:3 Changed 4 years ago by git

  • Commit changed from 4db20c0eb91d44268ace513ee2ba4071d80fb5b1 to 4e7d27a4cfcf1b0a8d591fbd46c98e66ec8bf2ce

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

4e7d27aMerge branch 'develop' into t/22465/doctest-continuation-last

comment:4 Changed 4 years ago by saraedum

  • Reviewers set to Julian Rüth
  • Status changed from needs_review to positive_review

I am happy with this, assuming that the patchbots do not complain.

comment:5 Changed 4 years ago by saraedum

  • Keywords days85 added

comment:6 Changed 4 years ago by vbraun

  • Branch changed from u/jmantysalo/doctest-continuation-last to 4e7d27a4cfcf1b0a8d591fbd46c98e66ec8bf2ce
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.