Skip to content

Commit e7d2570

Browse files
author
Release Manager
committed
gh-36183: `sage.misc.sageinspect`: fix doctest warnings due to modularisation We fix the following doctest warnings: ``` sage -t --long --warn-long 30.7 --random- seed=263240816384135969928803999009622073573 src/sage/misc/sageinspect.py ********************************************************************** File "src/sage/misc/sageinspect.py", line 1627, in sage.misc.sageinspect.sage_getargspec Warning: Variable 'Foo' referenced here was set only in doctest marked '# needs sage.misc.cython' sage_getargspec(Foo.join) ********************************************************************** File "src/sage/misc/sageinspect.py", line 1629, in sage.misc.sageinspect.sage_getargspec Warning: Variable 'Bar' referenced here was set only in doctest marked '# needs sage.misc.cython' sage_getargspec(Bar.join) ********************************************************************** File "src/sage/misc/sageinspect.py", line 1631, in sage.misc.sageinspect.sage_getargspec Warning: Variable 'Bar' referenced here was set only in doctest marked '# needs sage.misc.cython' sage_getargspec(Bar.meet) [353 tests, 32.04 s] ``` These are (currently) the only warnings I see when running `./sage -t --long src/sage/misc/`. ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> <!-- If your change requires a documentation PR, please link it appropriately --> <!-- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> <!-- Feel free to remove irrelevant items. --> - [x] The title is concise, informative, and self-explanatory. - [x] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation accordingly. ### ⌛ Dependencies <!-- List all open PRs that this PR logically depends on - #12345: short description why this is a dependency - #34567: ... --> <!-- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> URL: #36183 Reported by: David Coudert Reviewer(s): Frédéric Chapoton
2 parents 7b5ac02 + bc7e9cf commit e7d2570

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

src/sage/misc/sageinspect.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1614,7 +1614,8 @@ def foo(x, a='\')"', b={not (2+1==3):'bar'}): return
16141614
16151615
The following was fixed in :trac:`16309`::
16161616
1617-
sage: cython( # needs sage.misc.cython
1617+
sage: # needs sage.misc.cython
1618+
sage: cython(
16181619
....: '''
16191620
....: class Foo:
16201621
....: @staticmethod

0 commit comments

Comments
 (0)