Skip to content

Commit f808a97

Browse files
authored
Merge pull request #52 from zapta/main
Fixed the pip packages uninstall action.
2 parents 1e1a91f + fe81f8f commit f808a97

2 files changed

Lines changed: 14 additions & 4 deletions

File tree

.github/actions/uninstall-all-pip-packages/action.yaml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ runs:
77
- name: Uninstall all pip packages
88
shell: bash
99
run: |
10-
#!/bin/bash
10+
set -x
1111
set -euo pipefail
1212
1313
echo "Listing installed packages before uninstallation..."
@@ -26,8 +26,9 @@ runs:
2626
else
2727
echo "Uninstalling the following packages:"
2828
echo "$package_list"
29-
# Use process substitution to feed the list as a "requirements file"
30-
pip uninstall -y -r <(echo "$package_list")
29+
# # Use process substitution to feed the list as a "requirements file"
30+
# pip uninstall -y -r <(echo "$package_list")
31+
echo "$package_list" | xargs pip uninstall -y
3132
fi
3233
3334
echo "Verifying that no packages remain..."

.github/workflows/test.yaml

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -403,7 +403,16 @@ jobs:
403403
404404
# -- Test action uninstall-all-pip-packages
405405
test-uninstall-all-pip-packages:
406-
runs-on: ubuntu-latest
406+
strategy:
407+
fail-fast: false
408+
matrix:
409+
os: [macos-latest, macos-15-intel, ubuntu-latest, windows-latest]
410+
411+
runs-on: ${{ matrix.os }}
412+
413+
defaults:
414+
run:
415+
shell: bash
407416

408417
steps:
409418
- name: Checkout this repo

0 commit comments

Comments
 (0)