Skip to content

Commit 937fc77

Browse files
committed
bug fixing
1 parent 015acf4 commit 937fc77

1 file changed

Lines changed: 13 additions & 6 deletions

File tree

.github/workflows/test.yml

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -82,12 +82,6 @@ jobs:
8282
with:
8383
python-version: ${{ matrix.python-version }}
8484

85-
- name: Checkout server-api and patch Docker path
86-
shell: bash
87-
run: |
88-
git clone --depth 1 https://github.com/openml/server-api.git server-api
89-
sed -i 's|\.\./server-api|./server-api|g' docker-compose.yml
90-
9185
- name: Install test dependencies, scikit-learn, and optional pandas
9286
shell: bash
9387
run: |
@@ -107,6 +101,12 @@ jobs:
107101
echo "BEFORE=$git_status" >> $GITHUB_ENV
108102
echo "Repository status before tests: $git_status"
109103
104+
- name: Checkout server-api and patch Docker path
105+
shell: bash
106+
run: |
107+
git clone --depth 1 https://github.com/openml/server-api.git server-api
108+
sed -i 's|\.\./server-api|./server-api|g' docker-compose.yml
109+
110110
- name: Show installed dependencies
111111
run: python -m pip list
112112

@@ -145,6 +145,13 @@ jobs:
145145
run: | # we need a separate step because of the bash-specific if-statement in the previous one.
146146
pytest -n 4 --durations=20 --dist load -sv --reruns 5 --reruns-delay 1 -m "not uses_test_server"
147147
148+
- name: Cleanup Docker setup
149+
if: always()
150+
shell: bash
151+
run: |
152+
rm -rf server-api
153+
git checkout docker-compose.yml
154+
148155
- name: Check for files left behind by test
149156
if: matrix.os != 'windows-latest' && always()
150157
run: |

0 commit comments

Comments
 (0)