Repository navigation
Resolve overload links through the primary definition - #147
Open
vinsymoyya wants to merge 1 commit into
Open
vinsymoyya wants to merge 1 commit into
vinsymoyya wants to merge 1 commit into
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Linking to an overloaded function or command currently raises
Found multiple ...s named ..., even though the extra blocks are usage variants of a single definition. This prevents help generation when an inline directive references that definition.Exclude secondary usage blocks, which deliberately have no tag, from tag lookup candidates. Resolve the primary definition's tag while retaining the existing error for genuinely duplicate definitions. Regression tests exercise function links, both command-link spellings, and real duplicates through parsed documentation blocks.
Addresses the link-resolution bug in #38. The separate signature-validation behavior discussed in that issue is outside this patch.
Validation: function and command regressions fail on upstream
b2b9273with the reported duplicate-definition exception.python -m pytest -q --doctest-modulespasses all 12 tests/doctest modules with the fix;git diff --checkpasses. The source checkout emits the existing setuptools-scm version warning.