docs, ci: use conda env in readthedocs #210
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
gadomski
added a commit
that referenced
this pull request
Nov 16, 2021
gadomski
added a commit
that referenced
this pull request
Nov 16, 2021
gadomski
force-pushed
the
issues/207-fix-docs
branch
from
c19fcff to
da426d6
Compare
gadomski
added a commit
that referenced
this pull request
Nov 16, 2021
gadomski
force-pushed
the
issues/207-fix-docs
branch
from
da426d6 to
f0a950c
Compare
gadomski
added a commit
that referenced
this pull request
Nov 16, 2021
gadomski
force-pushed
the
issues/207-fix-docs
branch
from
f0a950c to
14766cd
Compare
gadomski
added a commit
that referenced
this pull request
Nov 16, 2021
gadomski
force-pushed
the
issues/207-fix-docs
branch
from
14766cd to
11e5271
Compare
|
nb: as part of this PR testing, I've enabled readthedocs builds as a part of PR CI. |
gadomski
added a commit
that referenced
this pull request
Nov 16, 2021
gadomski
force-pushed
the
issues/207-fix-docs
branch
from
11e5271 to
6d2a9a9
Compare
gadomski
added a commit
that referenced
this pull request
Nov 16, 2021
gadomski
force-pushed
the
issues/207-fix-docs
branch
from
6d2a9a9 to
96f2fc9
Compare
gadomski
added a commit
that referenced
this pull request
Nov 16, 2021
gadomski
force-pushed
the
issues/207-fix-docs
branch
from
96f2fc9 to
916c01b
Compare
gadomski
added a commit
that referenced
this pull request
Nov 16, 2021
gadomski
force-pushed
the
issues/207-fix-docs
branch
from
916c01b to
8f17918
Compare
gadomski
added a commit
that referenced
this pull request
Nov 16, 2021
gadomski
force-pushed
the
issues/207-fix-docs
branch
from
d6c7c16 to
a6d5111
Compare
gadomski
added a commit
that referenced
this pull request
Nov 16, 2021
gadomski
force-pushed
the
issues/207-fix-docs
branch
from
a6d5111 to
aefaa6c
Compare
Add GDAL to our ReadTheDocs build. Adds a RTD environment file to use in ReadTheDocs because you can't combine conda and pip deps in the readthedocs config. Fixes #207.
gadomski
force-pushed
the
issues/207-fix-docs
branch
from
aefaa6c to
99d607d
Compare
gadomski
requested review from
cuttlefish and
matthewhanson
and removed request for
cuttlefish
matthewhanson
approved these changes
Nov 18, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Related Issue(s): #207
Description: Installs our entire environment via a readthedocs-specific environment file. Readthedoc can't do both conda and pip installs, so we have to do everything via conda to get
gdalin the build.PR checklist:
scripts/format).scripts/lint).scripts/test).