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

Make :any: work with unknown object types in intersphinx. #5598

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sir-sigurd
Copy link

Subject: Make :any: work with unknown object types in intersphinx

Feature or Bugfix

Feature

Purpose

As suggested in #5562 (comment), if you want to refer to some custom object type defined in foreign project you had to define it in your project which is inconvenient. It's needed because intersphinx searches foreign inventory only for registered object types. This PR makes :any: role to not limit targets by registered object types.

Relates

Copy link
Member

@tk0miya tk0miya left a comment

Choose a reason for hiding this comment

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

It seems tests are broken. Could you check it please?

@codecov
Copy link

codecov bot commented Nov 12, 2018

Codecov Report

Merging #5598 into master will increase coverage by 0.02%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5598      +/-   ##
==========================================
+ Coverage   83.22%   83.24%   +0.02%     
==========================================
  Files         290      290              
  Lines       39345    39365      +20     
  Branches     5872     5872              
==========================================
+ Hits        32744    32769      +25     
+ Misses       5247     5245       -2     
+ Partials     1354     1351       -3
Impacted Files Coverage Δ
tests/test_util_inventory.py 100% <ø> (ø) ⬆️
sphinx/ext/intersphinx.py 90.54% <100%> (+2.25%) ⬆️
tests/test_ext_intersphinx.py 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f6ac360...e22b183. Read the comment docs.

@sir-sigurd
Copy link
Author

@tk0miya, tests are fixed now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants