Skip to content

Commit 67b4623

Browse files
authored
Run all python tests, they no longer depend on a token (#69)
* Run all python tests, they no longer depend on a token * Add [test] deps * sklearn -> scikit-learn * Add transitive dependency
1 parent 8d6ad65 commit 67b4623

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

.github/workflows/ci_python.yml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,8 +28,9 @@ jobs:
2828
run: |
2929
cd apis/python
3030
pip install .[test]
31-
pytest -k "not ingest" # TODO: requires token
31+
pytest
32+
# TODO: fix editable on linux
3233
#pip uninstall -y tiledb.vector_search
3334
#pip install -e .
34-
#pytest -k "not ingest" # TODO: requires token
35+
#pytest
3536
shell: bash -el {0}

apis/python/pyproject.toml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,10 +20,11 @@ classifiers = [
2020
dependencies = [
2121
"tiledb-cloud>=0.10.5",
2222
"tiledb>=0.15.2",
23+
"typing-extensions" # for tiledb-cloud indirect
2324
]
2425

2526
[project.optional-dependencies]
26-
test = ["pytest"]
27+
test = ["pytest", "scikit-learn", "tiledb-cloud"]
2728

2829

2930
[project.urls]

0 commit comments

Comments
 (0)