Skip to content

Commit 4666168

Browse files
committed
remove pipx from gh build
1 parent 6bc58c9 commit 4666168

File tree

1 file changed

+3
-13
lines changed

1 file changed

+3
-13
lines changed

.github/workflows/build.yml

Lines changed: 3 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,6 @@ jobs:
4444
matrix:
4545
os: [ubuntu-latest, macos-15-intel] # this is the last intel build, will be deprecated dec 2027, and atm we can't run on non-intel macs: https://github.com/psathyrella/partis/issues/330
4646
python-version: ['3.12']
47-
install-method: ['pip', 'pipx']
4847
fail-fast: false
4948

5049
env:
@@ -56,12 +55,10 @@ jobs:
5655
fetch-depth: 0
5756

5857
- name: Checkout ham submodule
59-
if: matrix.install-method == 'pip'
6058
run: |
6159
git submodule update --init packages/ham
6260
6361
- name: Checkout ig-sw submodule
64-
if: matrix.install-method == 'pip'
6562
run: |
6663
git submodule update --init packages/ig-sw
6764
@@ -74,7 +71,7 @@ jobs:
7471
if: matrix.os == 'ubuntu-latest'
7572
run: |
7673
sudo apt update
77-
sudo apt install -y python3 python3-pip python-is-python3 pipx build-essential cmake libgsl-dev libyaml-cpp-dev scons mafft ncurses-base ncurses-bin r-base
74+
sudo apt install -y python3 python3-pip python-is-python3 build-essential cmake libgsl-dev libyaml-cpp-dev scons mafft ncurses-base ncurses-bin r-base
7875
# sudo apt install libbpp-core libbpp-seq # which packages do i actually have to write here?
7976
8077
- name: Install system dependencies (macOS)
@@ -96,16 +93,10 @@ jobs:
9693
# R --slave -e 'dir.create(c(Sys.getenv("R_LIBS_USER")), recursive=TRUE); install.packages(c("TreeSim", "TreeSimGM", "geiger", "MASS"), repos="http://cran.rstudio.com/", dependencies=TRUE, lib=c(Sys.getenv("R_LIBS_USER")))'
9794

9895
- name: Install with pip
99-
if: matrix.install-method == 'pip'
10096
run: |
10197
python -m pip install --upgrade pip
10298
python -m pip install -e . -v
10399
104-
- name: Install with pipx
105-
if: matrix.install-method == 'pipx'
106-
run: |
107-
pipx install partis-bcr
108-
109100
- name: Test partis help
110101
run: |
111102
partis --help
@@ -120,7 +111,6 @@ jobs:
120111
121112
- name: Run full test
122113
run: |
123-
# if [[ "${{ matrix.install-method }}" == "pipx" || "${{ matrix.os }}" == macos* ]]; then
124114
if [[ "${{ matrix.os }}" == macos* ]]; then
125115
partis-test.py --no-simu
126116
partis-test.py --paired --no-simu
@@ -130,15 +120,15 @@ jobs:
130120
fi
131121
132122
- name: Verify tag is on main
133-
if: matrix.install-method == 'pip' && matrix.os == 'ubuntu-latest' && startsWith(github.ref, 'refs/tags/v')
123+
if: matrix.os == 'ubuntu-latest' && startsWith(github.ref, 'refs/tags/v')
134124
run: |
135125
if ! git branch -r --contains HEAD | grep -q 'origin/main'; then
136126
echo "ERROR: tag is not on main branch, skipping PyPI publish"
137127
exit 1
138128
fi
139129
140130
- name: Build and publish to PyPI
141-
if: matrix.install-method == 'pip' && matrix.os == 'ubuntu-latest' && startsWith(github.ref, 'refs/tags/v')
131+
if: matrix.os == 'ubuntu-latest' && startsWith(github.ref, 'refs/tags/v')
142132
run: |
143133
python -m pip install build twine
144134
python -m build --sdist 2>&1 | grep -v 'adding .test/\|adding .data/\|creating partis_bcr.*/test/\|copying docs/\|copying data/\|copying packages/\|copying python/.*/test/\|copying build/.*/test/'

0 commit comments

Comments
 (0)