search

ansys / ansys-sphinx-theme

PyData-based Sphinx theme from the PyAnsys community
https://sphinxdocs.ansys.com
MIT License
22 stars 5 forks source link

'if' condition on 'uri' not needed #298

Closed MaxJPRey closed 10 months ago

MaxJPRey commented 10 months ago

I think that this if condition is not needed as we are testing it few line below.

Or you could remove the second part of the condition statement below or not uri.

I might be missing a point though. The purpose of this issue is to make sure that it is intended.

https://github.com/ansys/ansys-sphinx-theme/blob/d9fe77c8e71f480109ad0eed630e86289091d2a1/src/ansys_sphinx_theme/extension/linkcode.py#L237