Skip to content

Commit

Permalink
add dagster-azure back to dev_install
Browse files Browse the repository at this point in the history
Summary:
#3573

#2483 (comment)

Test Plan:
in fresh virtualenv
```
~/dev/dagster arcpatch-D6207
$ make dev_install

~/dev/dagster arcpatch-D6207
$ test-D6207 $ pip freeze | grep azure
azure-common==1.1.26
azure-core==1.10.0
azure-storage-blob==12.3.2
azure-storage-file-datalake==12.0.2
-e [email protected]:dagster-io/dagster.git@bd35c74ff476078799a55650d70fa5c28b43d373#egg=dagster_azure&subdirectory=python_modules/libraries/dagster-azure

~/dev/dagster/docs arcpatch-D6207
test-D6207 $ make buildnext && cd next && yarn && yarn dev
```
{F536160}

Reviewers: sashank, nate, sandyryza

Reviewed By: nate, sandyryza

Differential Revision: https://dagster.phacility.com/D6207
  • Loading branch information
yuhan committed Feb 2, 2021
1 parent c27142f commit 78af868
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions scripts/install_dev_python_modules.py
Original file line number Diff line number Diff line change
Expand Up @@ -84,14 +84,10 @@ def main(quiet):
"-e python_modules/libraries/lakehouse",
"-e integration_tests/python_modules/dagster-k8s-test-infra",
"-r scala_modules/scripts/requirements.txt",
"-e python_modules/libraries/dagster-azure",
#
# https://github.com/dagster-io/dagster/issues/3488
# "-e python_modules/libraries/dagster-airflow",
#
# https://github.com/dagster-io/dagster/pull/2483#issuecomment-635174157
# Uncomment only when snowflake-connector-python can be installed with optional (or
# compatible) Azure dependencies.
# "-e python_modules/libraries/dagster-azure",
]

# dagster-ge depends on a great_expectations version that does not install on Windows
Expand Down

0 comments on commit 78af868

Please sign in to comment.