Skip to content

Commit a82d725

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

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

tox.ini

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -40,11 +40,17 @@ deps =
4040

4141
allowlist_externals = bash, sed
4242

43-
commands =
44-
pip freeze
4543

44+
install_command =
4645
# lsdb has tighter minimum dependencies, deal with it here for now, long term handle it from the notebook metadata
47-
oldestdeps: bash -c 'echo tutorials/parquet-catalog-demos/irsa-hats-with-lsdb >> ignore_testing'
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+
commands =
53+
pip freeze
4854

4955
# Ignore testing the tutorials listed in ignore_testing file. We have some OS specific ignores,
5056
# too due to issues with e.g. multiprocessing and problems in upstream dependency

0 commit comments

Comments
 (0)