Skip to content

Commit

Permalink
fix dbt dependency updated; remove extra python setup call
Browse files Browse the repository at this point in the history
  • Loading branch information
mikealfare committed Dec 10, 2024
1 parent 0273da4 commit bd0c4ab
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 8 deletions.
4 changes: 2 additions & 2 deletions .github/scripts/update_dependencies.sh
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,8 @@ set -e

git_branch=$1
target_req_file="hatch.toml"
core_req_sed_pattern="s|dbt-core.git#subdirectory=core|dbt-core.git@${git_branch}#subdirectory=core|g"
tests_req_sed_pattern="s|dbt-adapters.git#subdirectory=dbt-tests-adapter|dbt-adapters.git@${git_branch}#subdirectory=dbt-tests-adapter|g"
core_req_sed_pattern="s|dbt-core.git.*#subdirectory=core|dbt-core.git@${git_branch}#subdirectory=core|g"
tests_req_sed_pattern="s|dbt-adapters.git.*#subdirectory=dbt-tests-adapter|dbt-adapters.git@${git_branch}#subdirectory=dbt-tests-adapter|g"
if [[ "$OSTYPE" == darwin* ]]; then
# mac ships with a different version of sed that requires a delimiter arg
sed -i "" "$core_req_sed_pattern" $target_req_file
Expand Down
2 changes: 1 addition & 1 deletion .github/scripts/update_dev_dependency_branches.sh
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ dbt_adapters_branch=$1
dbt_core_branch=$2
dbt_common_branch=$3
target_req_file="hatch.toml"
core_req_sed_pattern="s|dbt-core.git#subdirectory=core|dbt-core.git@${dbt_core_branch}#subdirectory=core|g"
core_req_sed_pattern="s|dbt-core.git.*#subdirectory=core|dbt-core.git@${dbt_core_branch}#subdirectory=core|g"
adapters_req_sed_pattern="s|dbt-adapters.git|dbt-adapters.git@${dbt_adapters_branch}|g"
common_req_sed_pattern="s|dbt-common.git|dbt-common.git@${dbt_common_branch}|g"
if [[ "$OSTYPE" == darwin* ]]; then
Expand Down
7 changes: 2 additions & 5 deletions .github/workflows/integration.yml
Original file line number Diff line number Diff line change
Expand Up @@ -198,11 +198,8 @@ jobs:
${{ inputs.dbt_common_branch }}
cat hatch.toml
- uses: actions/setup-python@v5
with:
python-version: ${{ matrix.python-version }}

- uses: pypa/hatch@install
- name: Install python dependencies
uses: pypa/hatch@install

- run: hatch run integration-tests tests/functional --ddtrace
env:
Expand Down

0 comments on commit bd0c4ab

Please sign in to comment.