numpy / numpydoc

Numpy's Sphinx extensions
https://numpydoc.readthedocs.io/
Other
302 stars 161 forks source link

See Also Parser Recognizes Sphinx XREF #558

Open j9ac9k opened 5 months ago

j9ac9k commented 5 months ago

The See Also section raises a ValueError when it encounters a valid sphinx cross-reference such as

:meth:`QImage.save <QImage.save>`

This commit allows the regex to parse the sphinx target, which is the section between the angled brackets.

Author Note: This is my first contribution to a repo such as this, so I likely missed something, comments/nit-picks welcome.

stefanv commented 5 months ago

We should probably allow this, but it would need a test to prevent future regression.

j9ac9k commented 5 months ago

We should probably allow this, but it would need a test to prevent future regression.

Happy to add tests, I didn't notice if there was an existing test I should add or, or just roll my own thing, which I can do too, but would likely need some guidance on where it should go.