Skip to content

Commit d8a7dab

Browse files
committed
CI: more workaround to ensure we patch the requirements before trying to install them
1 parent 70eca8b commit d8a7dab

File tree

1 file changed

+10
-3
lines changed

1 file changed

+10
-3
lines changed

tox.ini

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,16 @@ deps =
4040

4141
allowlist_externals = bash, jupyter-book
4242

43+
44+
install_command =
45+
# lsdb has tighter minimum dependencies, deal with it here for now, long term handle it from the notebook metadata
46+
# We need to do this here before the dependencies are installed to work around deps conflicts
47+
oldestdeps: bash -c "echo tutorials/parquet-catalog-demos/irsa-hats-with-lsdb >> ignore_testing; sed -i -e 's|lsdb|\#lsdb|g' .binder/requirements.txt && python -I -m pip install $@"
48+
49+
# Adding back the default install command
50+
!oldestdeps: python -I -m pip install {opts} {packages}
51+
52+
4353
commands =
4454
pip freeze
4555

@@ -52,9 +62,6 @@ commands =
5262
bash -c 'if [[ $CI == true ]]; then cat ignore_tutorials/ignore_gha_testing >> ignore_testing; fi'
5363
bash -c 'if [[ $CIRCLECI == true ]]; then cat ignore_tutorials/ignore_circleci_testing >> ignore_testing; fi'
5464

55-
# lsdb has tighter minimum dependencies, deal with it here for now, long term handle it from the notebook metadata
56-
oldestdeps: bash -c 'echo tutorials/parquet-catalog-demos/irsa-hats-with-lsdb >> ignore_testing'
57-
5865
buildhtml: bash -c 'cat ignore_tutorials/ignore_rendering >> ignore_testing'
5966

6067
# We only want to run CI in PRs for the notebooks we touched

0 commit comments

Comments
 (0)