Page MenuHomePhabricator

fix read-the-docs reqs.txt
ClosedPublic

Authored by alangenfeld on Aug 7 2019, 9:44 PM.

Details

Summary

pretty sure this is what we want, and it is currently causing problems with the dev install script

Test Plan

???

Diff Detail

Repository
R1 dagster
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

alangenfeld created this revision.Aug 7 2019, 9:44 PM
alangenfeld added inline comments.Aug 7 2019, 9:46 PM
.read-the-docs-requirements.txt
3–5

i think with this previous setup there could be issues with the master version of the docs since it was installing public dagster/dagstermill

for reference this file is referred to in our config https://readthedocs.org/dashboard/dagster/advanced/

natekupp accepted this revision.Aug 7 2019, 10:49 PM
This revision is now accepted and ready to land.Aug 7 2019, 10:49 PM
This revision was automatically updated to reflect the committed changes.