Skip to content

Add CI test to mimic Read The Docs build #1445

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

Closed
JohnVillalovos opened this issue May 3, 2021 · 1 comment · Fixed by #3127
Closed

Add CI test to mimic Read The Docs build #1445

JohnVillalovos opened this issue May 3, 2021 · 1 comment · Fixed by #3127
Labels
enhancement test For issues or missing tests

Comments

@JohnVillalovos
Copy link
Member

JohnVillalovos commented May 3, 2021

See #1442

We had a change which broke the Read The Docs (RTD) build.

We should have a way to mimic the RTD build in our CI so that in the future we don't accidentally break the RTD builds. I'm not sure how to do that but hopefully someone else knows.

@nejch nejch added enhancement test For issues or missing tests labels May 30, 2021
@nejch
Copy link
Member

nejch commented Jul 23, 2022

Instead of adding another test, we could do configuration as code for RTD and define the steps to match our tox build:

https://docs.readthedocs.io/en/stable/config-file/v2.html
https://docs.readthedocs.io/en/stable/config-file/v2.html#build-commands

Low priority though IMO.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement test For issues or missing tests
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants