fix: Update RTD config. #418
Conversation
dianakhuang
force-pushed
the
diana/update-rtd-config
branch
from
56e10e6 to
0b3d881
Compare
dianakhuang
force-pushed
the
diana/update-rtd-config
branch
from
c60507b to
5eb7e4b
Compare
| - method: pip | ||
| path: . |
There was a problem hiding this comment.
@dianakhuang: Did you find any information about this, or are you just following what was done in edx-platform?
There was a problem hiding this comment.
I was just following what was done in edx-platform under the assumption that getting a working RTD config was more important than worrying about whether we may have redundant config.
There was a problem hiding this comment.
FYI, this is needed in edx-platform because we have to pip install . in order to add all the plugin entrypoints that edx-platform needs to properly start up django. Which we do to generate the module docs now. We shouldn't generally need this in other repos.
| os: "ubuntu-22.04" | ||
| tools: | ||
| python: "3.8" | ||
|
|
||
| python: | ||
| version: 3.8 |
There was a problem hiding this comment.
I think you want to remove this redundant version declaration.
RTD updated their config format. This updates the cookiecutter to match their recommendations. #416
dianakhuang
force-pushed
the
diana/update-rtd-config
branch
from
5eb7e4b to
3b1eb8c
Compare
|
Closing as a dupe of #419 |
RTD updated their config format. This updates
the cookiecutter to match their recommendations.
#416
Merge checklist:
Check off if complete or not applicable: