Add a hint in the automatic doctest sig_on_count()

[nathanncohen]

While working on #18746, I met a doctest error for a doctest that never appeared in my file. I looked at it for a while before undersdtanding what was happening. This branch adds the 'hint' I needed at that time.

Nathann

CC: @dcoudert

Component: doctest framework

Author: Nathann Cohen

Branch/Commit: 2e666dd

Reviewer: Jeroen Demeyer

Issue created by migration from https://trac.sagemath.org/ticket/18829

[nathanncohen]

Commit: b70e19e

[nathanncohen]

New commits:

b70e19e

trac #18829: Add a hint in the automatic doctest sig_on_count()

[nathanncohen]

Branch: u/ncohen/18829

[dcoudert]

comment:2

Shouldn't you set this patch to needs review?

I assume than for testing it we have no choice but doctesting the full src/sage/ directory. Right?

[nathanncohen]

comment:3

Agggggg, I am an idiot. I had started all tests when I created the branch, and this morning I closed a remote ssh session without knowing what it had been testing. It was probably this patch. I'll re-run the tests T_T

[sagetrac-git]

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

ca2980f	trac #18829: Merged with 6.8.beta7
d42c0a4	trac #18829: Broken doctests

[sagetrac-git]

Changed commit from b70e19e to d42c0a4

[jdemeyer]

comment:6

Can you just define a string constant for

"sig_on_count() # check sig_on/off pairings (virtual doctest)\n"

instead of writing the same string several times.

[nathanncohen]

comment:7

Done !

For this kind of trivial changes, it is not too daring for a reviewer to propose a commit ^^;

Nathann

[sagetrac-git]

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

2e666dd

trac #18829: One string to rule them all

[sagetrac-git]

Changed commit from d42c0a4 to 2e666dd

[jdemeyer]

Reviewer: Jeroen Demeyer

[vbraun]

Changed branch from u/ncohen/18829 to 2e666dd