Skip to content

Commit

Permalink
Update smoke-test.yml
Browse files Browse the repository at this point in the history
Fixes #101.

I already did this for the unit tests but didn't fix the smoke tests. Essentially we need to make sure that the workflow runs pip install -e .[dev] with the -e so that the repo's directory structure is maintained. If we don't include the -e then the unit tests can't find the testing database.
  • Loading branch information
astronomerritt authored Apr 11, 2024
1 parent 202a1e5 commit c598d27
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/smoke-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -32,8 +32,8 @@ jobs:
run: |
sudo apt-get update
python -m pip install --upgrade pip
pip install .
pip install .[dev]
pip install -e .
pip install -e .[dev]
if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
- name: List dependencies
run: |
Expand Down

0 comments on commit c598d27

Please sign in to comment.