Skip to content

Commit

Permalink
Remove superfluous --system arg to pip
Browse files Browse the repository at this point in the history
  • Loading branch information
Twixes committed Apr 10, 2024
1 parent ff22560 commit 5b7892e
Show file tree
Hide file tree
Showing 5 changed files with 14 additions and 14 deletions.
4 changes: 2 additions & 2 deletions .github/actions/run-backend-tests/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ runs:
- name: Install Python dependencies
shell: bash
run: |
pip install --system -r requirements.txt -r requirements-dev.txt
pip install -r requirements.txt -r requirements-dev.txt
- name: Install the working version of hogql-parser
if: steps.hogql-parser-diff.outputs.changed == 'true'
Expand All @@ -111,7 +111,7 @@ runs:
sudo cp out/usr/local/lib/libantlr4-runtime.so* /usr/lib/
sudo ldconfig
cd ..
pip install --system ./hogql_parser
pip install ./hogql_parser
- name: Set up needed files
shell: bash
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/benchmark.yml
Original file line number Diff line number Diff line change
Expand Up @@ -67,11 +67,11 @@ jobs:
- name: Install python dependencies
run: |
pip install --system -r requirements-dev.txt
pip install --system -r requirements.txt
pip install -r requirements-dev.txt
pip install -r requirements.txt
- name: Install asv
run: pip install --system asv==0.5.1 virtualenv
run: pip install asv==0.5.1 virtualenv

- name: Set up PostHog
run: |
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/build-hogql-parser.yml
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ jobs:
run: cd hogql_parser && python setup.py sdist

- name: Install cibuildwheel
run: pip install --system cibuildwheel==2.16.*
run: pip install cibuildwheel==2.16.*

- name: Build wheels
run: cd hogql_parser && python -m cibuildwheel --output-dir dist
Expand Down
8 changes: 4 additions & 4 deletions .github/workflows/ci-backend.yml
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,7 @@ jobs:
- name: Install Python dependencies
run: |
pip install --system -r requirements.txt -r requirements-dev.txt
pip install -r requirements.txt -r requirements-dev.txt
- name: Check for syntax errors, import sort, and code style violations
run: |
Expand Down Expand Up @@ -193,7 +193,7 @@ jobs:
- name: Install python dependencies
run: |
pip install --system -r requirements.txt -r requirements-dev.txt
pip install -r requirements.txt -r requirements-dev.txt
- uses: actions/checkout@v3
with:
Expand All @@ -203,7 +203,7 @@ jobs:
run: |
# We need to ensure we have requirements for the master branch
# now also, so we can run migrations up to master.
pip install --system -r requirements.txt -r requirements-dev.txt
pip install -r requirements.txt -r requirements-dev.txt
python manage.py migrate
- uses: actions/checkout@v3
Expand Down Expand Up @@ -342,7 +342,7 @@ jobs:
- name: Install python dependencies
shell: bash
run: |
pip install --system -r requirements.txt -r requirements-dev.txt
pip install -r requirements.txt -r requirements-dev.txt
- name: Add kafka host to /etc/hosts for kafka connectivity
run: sudo echo "127.0.0.1 kafka" | sudo tee -a /etc/hosts
Expand Down
8 changes: 4 additions & 4 deletions .github/workflows/ci-plugin-server.yml
Original file line number Diff line number Diff line change
Expand Up @@ -131,8 +131,8 @@ jobs:
- name: Install python dependencies
if: needs.changes.outputs.plugin-server == 'true'
run: |
pip install --system -r requirements-dev.txt
pip install --system -r requirements.txt
pip install -r requirements-dev.txt
pip install -r requirements.txt
- name: Install pnpm
if: needs.changes.outputs.plugin-server == 'true'
Expand Down Expand Up @@ -220,8 +220,8 @@ jobs:
- name: Install python dependencies
run: |
pip install --system -r requirements-dev.txt
pip install --system -r requirements.txt
pip install -r requirements-dev.txt
pip install -r requirements.txt
- name: Install pnpm
uses: pnpm/action-setup@v2
Expand Down

0 comments on commit 5b7892e

Please sign in to comment.