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

Change Pallets-Sphinx-Themes dependency to the latest commit #307

Merged
merged 1 commit into from Jan 3, 2020
Merged

Change Pallets-Sphinx-Themes dependency to the latest commit #307

merged 1 commit into from Jan 3, 2020

Conversation

alisianoi
Copy link
Contributor

@alisianoi alisianoi commented Jan 2, 2020

Edit: Upstream was very quick to push the fix, so updated this PR

There is an exception [1] if you try to execute make html in the docs
directory if you're using Python 3.8. This was addressed in the
Pallets-Sphinx-Themes repo but there has been no release since then.
Temporarily depend on the latest commit instead of PyPI version.

See: pallets/pallets-sphinx-themes@706ed73

[1]:

Exception occurred:
File "/home/alex/Code/pyenv/versions/3.8.0/lib/python3.8/importlib/metadata.py", line 381, in find_distributions
found = cls._search_paths(context.pattern, context.path)
AttributeError: 'str' object has no attribute 'pattern'
The full traceback has been saved in /tmp/sphinx-err-5ejlxhto.log, if you want to report the issue to the developers.
Please also report this if it was a user error, so that a better error message can be provided next time.
A bug report can be filed in the tracker at https://github.com/sphinx-doc/sphinx/issues. Thanks!
make: *** [Makefile:53: html] Error 2

@coveralls
Copy link

coveralls commented Jan 2, 2020

Coverage Status

Coverage remained the same at 100.0% when pulling d8deb6d on alisianoi:master into df430f6 on vimalloc:master.

Version 1.2.2 had a small issue with Python 3.8

See: pallets/pallets-sphinx-themes#27
@vimalloc vimalloc merged commit bf1a521 into vimalloc:master Jan 3, 2020
@vimalloc
Copy link
Owner

vimalloc commented Jan 3, 2020

Awesome, thanks! 👍

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

Successfully merging this pull request may close these issues.

None yet

3 participants