I have created a pull request for this issue <https://github.com/sagemath/sage/pull/38215>.
On Thursday 13 June 2024 at 11:22:54 UTC+2 Jens-Erik Riedel wrote: > Thanks to a hint by Dima I found the same typo in two other files. > In sage/structure/factory.pyx line 523 it prevents a doctest from being > run. > In sage/rings/polynomial/padics/polynomial_padic.py line 277 it is merely > cosmetic. > I will create a Pull Request with fixes for all three occurrences of this > typo. > > On Wednesday, June 12, 2024 at 4:46:45 PM UTC+2 julian...@gmail.com wrote: > >> Would you mind creating a Pull Request with these changes at >> https://github.com/sagemath/sage? >> >> julian >> >> On Wednesday, June 12, 2024 at 5:31:01 PM UTC+3 jenseri...@t-online.de >> wrote: >> >>> When running "sage -t" on the file >>> sage/modular/pollack_stevens/padic_lseries.py no doctests are executed >>> although the file contains quite a lot of doctests. >>> I think the reason for that is a typo in the very first line of the file: >>> # sage.doctest: needs sage.ring.padics >>> It must be "rings" instead of "ring. >>> This is in Sage 10.3 as well as in Sage 10.4.beta9. >>> There is another typo in line 177 of that file: In the doctest comment a >>> comma is missing between "indirect doctest" and "long time". >>> After having corrected these two typos all doctests in >>> sage/modular/pollack_stevens/padic_lseries.py are successfully executed. >>> >> -- You received this message because you are subscribed to the Google Groups "sage-support" group. To unsubscribe from this group and stop receiving emails from it, send an email to sage-support+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/sage-support/b59e4f92-8e45-4827-a948-97cb4eb59764n%40googlegroups.com.
