-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Possible path error? #831
Comments
You may be right, though I also think that option doesn't have any effect locally or on the RTD hosted version. Locally, we import the template after appending the local path to the theme ( Line 8 in ddf840c
On RTD, the theme is installed from PyPI, and so the Perhaps try removing the option and see if that breaks anything. My guess is it should work both locally and on RTD. |
I ended up testing this in #838 and the line is removed there. Change looks good so far |
Sorry I didn't get back to you on this. I did do some testing and as far as I could see it didn't cause any problems - but on the other hand I couldn't make it do anything terrible by having a wayward zip in the parent folder. Removing the line does stop that warning appearing though, so I think it's worth doing. |
Not sure if this is intentional or not but in this line:
sphinx_rtd_theme/docs/conf.py
Line 51 in ddf840c
html_theme_path
points to a directory outside the repo. So on my system when I build I get warnings likewhere
imlv.zip
just happens to be a zip file with the same parent directory assphinx_rtd_theme
.The text was updated successfully, but these errors were encountered: