Skip to content

Commit

Permalink
tested if dancing with venv activation/deactivation fixes the path re…
Browse files Browse the repository at this point in the history
…solution
  • Loading branch information
CommanderStorm committed Apr 28, 2024
1 parent 22bded1 commit 7a89179
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 0 deletions.
6 changes: 6 additions & 0 deletions .github/workflows/data-cicd.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,10 +23,13 @@ jobs:
- name: setup uv
run: curl -LsSf https://astral.sh/uv/install.sh | sh
- name: Install dependencies
# activating and deactiviating because of https://stackoverflow.com/questions/35045038/how-do-i-use-pytest-with-virtualenv
run: |
uv venv
source .venv/bin/activate
uv pip install -r data/requirements.txt -r requirements-dev.txt
deactivate
source .venv/bin/activate
- run: pytest data
#type-check:
# runs-on: ubuntu-latest
Expand All @@ -39,10 +42,13 @@ jobs:
# - name: setup uv
# run: curl -LsSf https://astral.sh/uv/install.sh | sh
# - name: Install dependencies
# # activating and deactiviating because of https://stackoverflow.com/questions/35045038/how-do-i-use-pytest-with-virtualenv
# run: |
# uv venv
# source .venv/bin/activate
# uv pip install -r data/requirements.txt -r requirements-dev.txt
# deactivate
# source .venv/bin/activate
# - name: Run mypy
# run: mypy --strict data
data-build:
Expand Down
3 changes: 3 additions & 0 deletions .github/workflows/linting.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,10 +28,13 @@ jobs:
- name: setup uv
run: curl -LsSf https://astral.sh/uv/install.sh | sh
- name: Install python dependencies
# activating and deactiviating because of https://stackoverflow.com/questions/35045038/how-do-i-use-pytest-with-virtualenv
run: |
uv venv
source .venv/bin/activate
uv pip install -r data/requirements.txt -r requirements-dev.txt
deactivate
source .venv/bin/activate
- name: Run pre-commit
continue-on-error: ${{ github.ref == 'refs/heads/main' }}
uses: pre-commit/[email protected]
Expand Down
3 changes: 3 additions & 0 deletions .github/workflows/update-data.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,10 +25,13 @@ jobs:
- name: setup uv
run: curl -LsSf https://astral.sh/uv/install.sh | sh
- name: Install python dependencies
# activating and deactiviating because of https://stackoverflow.com/questions/35045038/how-do-i-use-pytest-with-virtualenv
run: |
uv venv
source .venv/bin/activate
uv pip install -r data/requirements.txt -r requirements-dev.txt
deactivate
source .venv/bin/activate
- name: Download data
continue-on-error: true # a PR deleting all data will be created if this fails => fail obvious
run: |
Expand Down

0 comments on commit 7a89179

Please sign in to comment.