Apply qualname_overrides in more circumstances

[flying-sheep]

Fixes doc problems in scverse/anndata#1726

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (684834c) to head (9800e53).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff            @@
##              main      #191   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           15        15           
  Lines          456       465    +9     
=========================================
+ Hits           456       465    +9     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ilan-gold]

I think this update should be documented https://icb-scanpydoc.readthedocs-hosted.com/en/latest/scanpydoc.elegant_typehints.html#module-scanpydoc.elegant_typehints ? Or no?

Also, just to be clear, this enables qualname_overrides to interact with intersphinx mappings in a way that affects how the doc is fetched, and not just rendered? Is that right?

[flying-sheep]

I think this update should be documented

The docs already say “It defines a configuration value qualname_overrides for conf.py that overrides automatically created links”.

This PR just makes that work more reliably.

Also, just to be clear, this enables qualname_overrides to interact with intersphinx mappings in a way that affects how the doc is fetched, and not just rendered? Is that right?

Sphinx fires the missing-reference event each time Sphinx can’t resolve a reference. The reference gets passed a pending_xref node that it’s supposed to replace with a reference node (i.e. a link), or signal that it failed to do so.

When intersphinx is enabled, it has an event handler for that event. If it fails, other event handlers for that event with a higher priority value are run (yeah higher priority value means lower priority).

This PR adds a new event handler for missing-reference (if intersphinx is enabled) with a priority value that makes it run after intersphinx’s, handler. So with this PR, after intersphinx fails to resolve a reference, this PR’s code tries running intersphinx’s code again, this time with the value from qualname_overrides.