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
move doc build checks from travis to readthedocs PR checks #3850
Comments
@graingert i think it is a good idea. I would like to know what @Pierre-Sassoulas , @AWhetter and @PCManticore think about it. |
Sounds good to me too. |
steps that are needed: go here: https://readthedocs.org/dashboard/pylint/advanced/#id_external_builds_enabled and tick this box then screenshot the "Default settings" fromset, it should look something like this: I can use that to create a .readthedocs.yml file then go here https://readthedocs.org/dashboard/pytlint/integrations/create/ to double check the webhook is enabled |
@Pierre-Sassoulas or I can set this up, with admin access to the pylint repo and readthedocs project |
I'm a pylint-admin now, let's do this :D |
Currently builds are checked with tox/travis, but the readthedocs environment is different/more restrictive than travis.
It would be better to have readthedocs build and check the docs for PRs rather than travis: https://docs.readthedocs.io/en/stable/guides/autobuild-docs-for-pull-requests.html
It's probably worth leaving the docs test in tox for local development
The text was updated successfully, but these errors were encountered: