Skip to content

Use Sphinx helper to re-route missing references - #196

Merged
inducer merged 3 commits into
inducer:mainfrom
alexfikl:hack-sphinx
Jul 29, 2025
Merged

Use Sphinx helper to re-route missing references#196
inducer merged 3 commits into
inducer:mainfrom
alexfikl:hack-sphinx

Conversation

@alexfikl

@alexfikl alexfikl commented Jul 29, 2025

Copy link
Copy Markdown
Collaborator

@alexfikl
alexfikl marked this pull request as draft July 29, 2025 19:01
Comment thread doc/conf.py
Comment on lines 38 to 39
# Sphinx started complaining about these in 8.2.1(-ish)
# -AK, 2025-02-24
["py:class", r"TypeAliasForwardRef"],

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This still seems to stick around. At least in the missing-reference call, the node that corresponds to this just says TypeAliasForwardRef with no information about what it's forwarding to.. so we can't dig that out :\

@alexfikl
alexfikl force-pushed the hack-sphinx branch 3 times, most recently from 427a0a8 to 4ab96d2 Compare July 29, 2025 19:26
@alexfikl
alexfikl marked this pull request as ready for review July 29, 2025 19:28
@inducer
inducer enabled auto-merge (rebase) July 29, 2025 20:08
@inducer

inducer commented Jul 29, 2025

Copy link
Copy Markdown
Owner

Thx!

@inducer
inducer merged commit 2064100 into inducer:main Jul 29, 2025
11 checks passed
@alexfikl
alexfikl deleted the hack-sphinx branch July 29, 2025 20:12
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.

2 participants