Skip to content

Commit

Permalink
Allow sphinx resolution of Azure types again. (microsoft#937)
Browse files Browse the repository at this point in the history
# Pull Request

## Title

Allow sphinx resolution of Azure types again.

______________________________________________________________________

## Description

See Also:
- Quansight-Labs/intersphinx_registry#58
- microsoft#912 

______________________________________________________________________

## Type of Change

- 🛠️ Bug fix
- 📝 Documentation update

______________________________________________________________________
  • Loading branch information
bpkroth authored Jan 24, 2025
1 parent f2b3e81 commit ed2c0cb
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 8 deletions.
2 changes: 1 addition & 1 deletion doc/requirements.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
setuptools-scm>=8.1.0
sphinx
sphinx-autoapi
intersphinx_registry>=0.2410.14 # https://github.com/Quansight-Labs/intersphinx_registry/pull/41
intersphinx_registry>=0.2501.23 # https://github.com/Quansight-Labs/intersphinx_registry/pull/58
nbsphinx
jupyter_core>=4.11.2 # nbsphix dependency - addresses CVE-2022-39286
nbconvert
Expand Down
9 changes: 2 additions & 7 deletions doc/source/conf.py
Original file line number Diff line number Diff line change
Expand Up @@ -136,10 +136,8 @@ def is_on_github_actions():
intersphinx_mapping = get_intersphinx_mapping(
packages={
"asyncssh",
# Azure SDKs removed their intersphinx publishing.
# https://github.com/Azure/azure-sdk-for-python/issues/39316
# "azure-core",
# "azure-identity",
"azure-core",
"azure-identity",
"configspace",
"matplotlib",
"numpy",
Expand Down Expand Up @@ -235,9 +233,6 @@ def setup(app: SphinxApp) -> None:
nitpick_ignore_regex = [
# Ignore some external references that don't use sphinx for their docs.
(r"py:.*", r"flaml\..*"),
# Azure SDKs removed their intersphinx publishing.
# https://github.com/Azure/azure-sdk-for-python/issues/39316
(r"py:.*", r"azure\..*"),
]

# Which documents to include in the build.
Expand Down

0 comments on commit ed2c0cb

Please sign in to comment.