Add missing deps for docs-snippets and skip 36 in BK (#8048)

dagster-io · May 25, 2022 · 31e1f3d · 31e1f3d
1 parent 49aa991
commit 31e1f3d
Show file tree

Hide file tree

Showing 3 changed files with 36 additions and 8 deletions.
diff --git a/.buildkite/dagster-buildkite/dagster_buildkite/steps/packages.py b/.buildkite/dagster-buildkite/dagster_buildkite/steps/packages.py
@@ -327,6 +327,11 @@ def k8s_extra_cmds(version: str, _) -> List[str]:
         "examples/docs_snippets",
         pytest_extra_cmds=docs_snippets_extra_cmds,
         run_mypy=False,
+        unsupported_python_versions=[
+            # dependency on various 3.6 and 3.9-incompatible extension libs
+            AvailablePythonVersion.V3_6,
+            AvailablePythonVersion.V3_9,
+        ],
     ),
     PackageSpec(
         "examples/ge_example",

diff --git a/examples/docs_snippets/setup.py b/examples/docs_snippets/setup.py
@@ -16,13 +16,35 @@
             "Operating System :: OS Independent",
         ],
         packages=find_packages(exclude=["test"]),
-        install_requires=["dagster", "dagster-k8s"],
+        install_requires=[
+            "dagit",
+            "dagster",
+            "dagstermill",
+            "dagster-airflow",
+            "dagster-aws",
+            "dagster-celery",
+            "dagster-dbt",
+            "dagster-dask",
+            "dagster-gcp",
+            "dagster-graphql",
+            "dagster-k8s",
+            "dagster-postgres",
+            "dagster-slack",
+        ],
         extras_require={
             "full": [
-                "seaborn",
-                "scikit-learn",
+                "click",
+                "matplotlib",
                 "moto==1.3.16",
+                "numpy",
+                "pandas",
                 "pandera",
+                "pytest",
+                "requests",
+                "seaborn",
+                "scikit-learn",
+                "slack_sdk",
+                "snapshottest",
             ]
         },
     )
diff --git a/examples/docs_snippets/tox.ini b/examples/docs_snippets/tox.ini
@@ -11,16 +11,17 @@ passenv = CI_* COVERALLS_REPO_TOKEN POSTGRES_TEST_DB_HOST BUILDKITE
 deps =
   -e ../../python_modules/dagster[mypy,test]
   -e ../../python_modules/dagster-graphql
+  -e ../../python_modules/libraries/dagstermill
   -e ../../python_modules/libraries/dagster-airflow
-  -e ../../python_modules/libraries/dagster-postgres
   -e ../../python_modules/libraries/dagster-aws
-  -e ../../python_modules/libraries/dagster-pandas
   -e ../../python_modules/libraries/dagster-celery
-  -e ../../python_modules/libraries/dagster-dask
-  -e ../../python_modules/libraries/dagster-slack
-  -e ../../python_modules/libraries/dagstermill
   -e ../../python_modules/libraries/dagster-dbt
+  -e ../../python_modules/libraries/dagster-dask
+  -e ../../python_modules/libraries/dagster-gcp
   -e ../../python_modules/libraries/dagster-k8s
+  -e ../../python_modules/libraries/dagster-pandas
+  -e ../../python_modules/libraries/dagster-postgres
+  -e ../../python_modules/libraries/dagster-slack
   -e ../../python_modules/dagit
 allowlist_externals =
   /bin/bash