Skip to content

Annotate the list of strings correctly for Sphinx 8.2+ #23

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
May 8, 2025

Conversation

befeleme
Copy link
Contributor

@befeleme befeleme commented Apr 3, 2025

This also works with an older Sphinx, tested on 8.1.3.
Closes #21

This also works with an older Sphinx, tested on 8.1.3.
@befeleme
Copy link
Contributor Author

befeleme commented Apr 3, 2025

I can make it 3.8 compatible, but since that version of Python is past the support phase, maybe that's not needed anymore?

@danigm
Copy link

danigm commented Apr 24, 2025

I can confirm that this patch fixes the tests in openSUSE Tumbleweed package: https://build.opensuse.org/package/show/devel:languages:python/python-sphinxcontrib-apidoc

bmwiedemann pushed a commit to bmwiedemann/openSUSE that referenced this pull request Apr 24, 2025
…72431

https://build.opensuse.org/request/show/1272431
by user dgarcia + anag_factory
- Add upstream patch sphinx-82.patch to support Sphinx >= 8.2
  gh#sphinx-contrib/apidoc#23
@stephenfin
Copy link
Member

@befeleme Please feel free to drop Python 3.8 support and I'll happily merge and release this

stephenfin added 2 commits May 8, 2025 13:41
Signed-off-by: Stephen Finucane <[email protected]>
@stephenfin stephenfin merged commit 0ea83c4 into sphinx-contrib:master May 8, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[bug] Nested apidoc_excluded_paths default [[str]] causes a warning in Sphinx 8.2.0
3 participants