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

Remove sphinx CI test #292

Merged
merged 3 commits into from
Dec 18, 2021
Merged

Remove sphinx CI test #292

merged 3 commits into from
Dec 18, 2021

Conversation

adamjstewart
Copy link
Collaborator

@adamjstewart adamjstewart commented Dec 15, 2021

Read the Docs CI seems to be working for this repo now, so we don't need to test documentation building ourselves, it's already run by Read the Docs on every commit. This also lets us view the docs before a PR is merged by clicking a link. See https://docs.readthedocs.io/en/stable/pull-requests.html.

@adamjstewart adamjstewart added documentation Improvements or additions to documentation testing Continuous integration testing labels Dec 15, 2021
@adamjstewart adamjstewart added this to the 0.1.1 milestone Dec 15, 2021
@adamjstewart adamjstewart force-pushed the tests/rtd branch 3 times, most recently from 5f7fb0b to a858ecf Compare December 17, 2021 16:22
isaaccorley
isaaccorley previously approved these changes Dec 17, 2021
@calebrob6 calebrob6 merged commit 5143064 into main Dec 18, 2021
@calebrob6 calebrob6 deleted the tests/rtd branch December 18, 2021 00:55
adamjstewart added a commit that referenced this pull request Dec 19, 2021
* Remove sphinx CI test

* Move mypy back to tests section

* Fix bad rebase
yichiac pushed a commit to yichiac/torchgeo that referenced this pull request Apr 29, 2023
* Remove sphinx CI test

* Move mypy back to tests section

* Fix bad rebase
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation testing Continuous integration testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants