Skip to content

Commit

Permalink
clean up and add comments
Browse files Browse the repository at this point in the history
  • Loading branch information
emmyoop committed Feb 1, 2024
1 parent 4a0e899 commit 5a71d54
Showing 1 changed file with 10 additions and 7 deletions.
17 changes: 10 additions & 7 deletions .github/workflows/ci_dbt_core_testing.yml
Original file line number Diff line number Diff line change
Expand Up @@ -106,18 +106,21 @@ jobs:
with:
python-version: "3.11"

- name: "Install python tools & dependencies"
- name: "Install python tools"
run: |
python -m pip install --user --upgrade pip
python -m pip --version
python -m pip install -r dev-requirements.txt -r editable-requirements.txt
python -m pip install tox
tox --version
- name: "Check installed versions"
run: pip freeze
# tox takes care of installing the correct version of dbt-core dependencies but we need to
# install them first s that we can override the dbt-common branch
- name: "Manually install dbt-core dependencies"
run: |
python -m pip install -r dev-requirements.txt -r editable-requirements.txt
- name: "Update the version of dbt-common@${{ inputs.dbt-common-ref }}"
# 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@${{ inputs.dbt-common-ref }}"
run: |
pip install pip install git+https://github.com/dbt-labs/dbt-common.git@${{ inputs.dbt-common-ref }} --force-reinstall
Expand Down

0 comments on commit 5a71d54

Please sign in to comment.