Skip to content

Commit

Permalink
sagemathgh-36183: sage.misc.sageinspect: fix doctest warnings due t…
Browse files Browse the repository at this point in the history
…o 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
- sagemath#12345: short description why this is a dependency
- sagemath#34567: ...
-->

<!-- If you're unsure about any of these, don't hesitate to ask. We're
here to help! -->

URL: sagemath#36183
Reported by: David Coudert
Reviewer(s): Frédéric Chapoton
  • Loading branch information
Release Manager committed Sep 4, 2023
2 parents 6348eb4 + bc7e9cf commit cc428be
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 5 deletions.
6 changes: 3 additions & 3 deletions build/pkgs/configure/checksums.ini
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
tarball=configure-VERSION.tar.gz
sha1=80ee52a3b6855149693204f0816bd0690947e7ae
md5=4292c175bba9299bd057980cec7060fd
cksum=3914108326
sha1=98c699503f9bf853cbdb54ddb8d48abfdb3c1113
md5=44ef95409d51e4ffde1aee8e7ace5f90
cksum=1323514555
2 changes: 1 addition & 1 deletion build/pkgs/configure/package-version.txt
Original file line number Diff line number Diff line change
@@ -1 +1 @@
57852a89c5c274ec6dbdd9269042972c7343e3ec
b90c4f2be83b5de6b4d1dbf92ec11c9ac7553bb0
3 changes: 2 additions & 1 deletion src/sage/misc/sageinspect.py
Original file line number Diff line number Diff line change
Expand Up @@ -1614,7 +1614,8 @@ def foo(x, a='\')"', b={not (2+1==3):'bar'}): return
The following was fixed in :trac:`16309`::
sage: cython( # needs sage.misc.cython
sage: # needs sage.misc.cython
sage: cython(
....: '''
....: class Foo:
....: @staticmethod
Expand Down

0 comments on commit cc428be

Please sign in to comment.