Skip to content

Commit

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

dagster-io#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 git+git@github.com: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 authored and David Laing committed Feb 20, 2021
1 parent 09023a7 commit 3757ded
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions scripts/install_dev_python_modules.py
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 3757ded

Please sign in to comment.