Skip to content

Commit

Permalink
fix pip install
Browse files Browse the repository at this point in the history
  • Loading branch information
emmyoop committed Feb 1, 2024
1 parent cbe2d0a commit fa3b167
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions .github/workflows/ci_dbt_core_testing.yml
Original file line number Diff line number Diff line change
Expand Up @@ -131,14 +131,15 @@ jobs:
# install them first s that we can override the dbt-common branch
- name: "Manually install dbt-core dependencies"
run: |
python -m pip uninstall dbt-common -y
python -m pip install -r dev-requirements.txt -r editable-requirements.txt
# Since the dbt-common dependency is pinned in dev-requirements.txt we need to force update it
# Since tox installs dependencies but doesn't force update, it won't get overridden in the next
# step since the requirements will already be met
- name: "Force update the version of dbt-common@${{ needs.job-prep.outputs.dbt-common-ref }}"
run: |
pip install pip install git+https://github.com/dbt-labs/dbt-common.git@${{ needs.job-prep.outputs.dbt-common-ref }} --force-reinstall
python -m pip install pip install git+https://github.com/dbt-labs/dbt-common.git@${{ needs.job-prep.outputs.dbt-common-ref }} --force-reinstall
- name: Run unit tests
run: tox
Expand Down Expand Up @@ -244,7 +245,8 @@ jobs:
# step since the requirements will already be met
- name: "Force update the version of dbt-common@${{ needs.job-prep.outputs.dbt-common-ref }}"
run: |
pip install pip install git+https://github.com/dbt-labs/dbt-common.git@${{ needs.job-prep.outputs.dbt-common-ref }} --force-reinstall
python -m pip uninstall dbt-common -y
python -m pip install pip install git+https://github.com/dbt-labs/dbt-common.git@${{ needs.job-prep.outputs.dbt-common-ref }} --force-reinstall
- name: Run Functional tests
run: tox
Expand Down Expand Up @@ -294,4 +296,4 @@ jobs:
- name: "Integration Tests Skipped"
if: ${{ github.event_name == 'pull_request' }}
run: |
echo "::notice title='Integration test suite skipped on Pull Requests'"
echo "::notice title='Integration test suite skipped on Pull Requests - they will run on merge'"

0 comments on commit fa3b167

Please sign in to comment.