-
Notifications
You must be signed in to change notification settings - Fork 56
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
Update Pipfile #290
Update Pipfile #290
Conversation
@dbu I don't think the CI process is working. Is Travis even required? The last PR I made (#288) seem to deploy just fine. The ability to preview changes doesn't seem to be tied to Travis either - having this enabled would cover most of the needs for having a CI process in a documentation project like this. |
thanks, i managed to make readthedocs build the branch. it seems to build with these changes, but the search is still broken: https://php-http--290.org.readthedocs.build/en/290/ the log of the build seems to be here: https://readthedocs.org/api/v2/build/14728001.txt if you have any idea, i appreciate input. otherwise i will try to look into it but can't promise when i get to it. |
@dbu RTD is using
It doesn't seem like RTD has ever supported Pipenv from the start: readthedocs/readthedocs.org#3181 The version of PIP used in RTD should support Poetry via PEP 517, if you are so inclined, or we could just use |
7cf741c
to
e9b6e08
Compare
593faf8
to
9cbc11f
Compare
thanks for the ping-pong! i think i found a working solution with recent versions of the stuff. and the search seems fixed \o/ |
the search on readthedocs is broken and there is a jquery error when loading the page.
we seem to use very old versions of sphinx and its dependencies. lets see if upgrading helps.
fix #289