-
Notifications
You must be signed in to change notification settings - Fork 180
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
Documentation not showing on readthedocs? #39
Comments
Interesting, my cached version still had the docs, so it's a recent-ish problem. Taking a look... |
Looks like eth-utils is using some py3.6 syntax. I'm investigating why that didn't fail in CI tests on py35. From the docs build:
|
This 💥ing thing again: ethereum/web3.py#1012 (comment) RTD must be using py3.5.2 as the default py3 |
Looks like we may be able to update to use the latest 3.0 builder: which uses 3.5.4 |
Which also looks like it has better support for how we specify our documentation dependencies. https://docs.readthedocs.io/en/latest/yaml-config.html#python-extra-requirements |
I'm blind, how do you know that 3.0 builder uses 3.5.4? https://docs.readthedocs.io/en/latest/builds.html#latest BTW, I'm waiting for RTD to pick up a couple new branch on upstream, so I can test it without merging a bunch of test PRs:
(waiting for them to show here https://readthedocs.org/dashboard/eth-account/versions/ ) |
Sorry, should have posted the "more details" link: https://github.com/rtfd/readthedocs-docker-images/blob/releases/3.x/Dockerfile |
What was wrong?
Code that produced the error
When running the docs locally
I see the full documentation for methods and whatnot, but on readthedocs I don't see any of that?
https://eth-account.readthedocs.io/en/latest/eth_account.html
Expected Result
Method and class doc shows up at the link above.
Environment
How can it be fixed?
N/A
The text was updated successfully, but these errors were encountered: