Skip to content
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

Fix #9630: autosummary: Failed to build summary table if primary_domain is not 'py' #9633

Merged
merged 2 commits into from Sep 17, 2021

Conversation

tk0miya
Copy link
Member

@tk0miya tk0miya commented Sep 14, 2021

Feature or Bugfix

  • Bugfix

Purpose

… is not 'py'

Autodoc generates reST code that uses raw `:obj:` and `:class:` xrefs to
refer the classes and types.  But they're fragile because they assume
the primary_domain=='py'.

This adds `:py:` prefix to these xrefs to make them robust.
…imary_domain is not 'py'

Autosummary generates reST code that uses raw `:obj:` xrefs to refer
the python objects in the summary table.  But they're fragile because
they assume the primary_domain=='py'.

This adds `:py:` prefix to these xrefs to make them robust.
@tk0miya tk0miya force-pushed the 9630_autosummary_primary_domain branch from 795659c to bc01207 Compare September 14, 2021 14:59
@tk0miya tk0miya merged commit bebd6bd into sphinx-doc:4.x Sep 17, 2021
@tk0miya tk0miya deleted the 9630_autosummary_primary_domain branch September 17, 2021 16:02
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant