TST: Enable doctest for dataframe.rst #19716

Activity

1. pllim

   added

   testing

   table

   Docs

   on May 8, 2026
2. RadeshL commented on May 17, 2026

   

   RadeshL

   on May 17, 2026

   Last edited by RadeshL

   More actions

   hey I would love to contribute to this issue, can I be assigned with it?

   My proposed solution:

   1. Remove ..doctest-skip-all from the file
   2. Add ..doctest-requires :: dierectives per block
   3. Preserve continuity of state of the block by consolidating dependent statements into self contained doctest blocks
3. arsenelupin14 commented on May 23, 2026

   

   arsenelupin14

   on May 23, 2026

   Contributor

   More actions

   Hi. I noticed this issue is still open.
   I willing to address the problem with the .. doctest-skip-all and i have already removed it and applied ..doctest-requires:: for pandas and polars especially to the @blocks

   Other couple of solutions that i implement:

   1. Keeping the original code blocks separated rather than merging them into 1 block (Sphinx naturally maintains the state across the blocks).
   2. Added a .. doctest-skip:: specifically for the Excel example ( pandas.read_excel ) to prevent CI failures since the illustrative myexceltable.xlsx (file isn't existed yet on the repository.

   But this approach had a two trade off:

   1. Untested Excel Snippet: By using .. doctest-skip:: for the read_excel, specifically snippet is now a gap in the CI.
   2. Pandas string/nullable representations (like , string8, str1) might cause the doctest to fail if the docs text is slightly outdated compared to the CI environment.
4. pllim

   added a commit that references this issue on May 28, 2026

   TST: Enable doctest for dataframes.rst (astropy#19716)

   f04bfe1
5. pllim

   closed this as completedin #19804on May 29, 2026
6. pllim

   added a commit that references this issue on May 29, 2026

   Merge pull request #19804 from arsenelupin14/fix-doctest-19716

   ...

   c373cc2