Skip to content

Commit d398702

Browse files
refactor: move disable_dbt_artifacts_autoupload to dbt_project.yml
Both vars now in dbt_project.yml, no --vars needed on CLI. disable_dbt_artifacts_autoupload only gates the on-run-end hook's upload_dbt_artifacts() call, so it's safe to set globally. Co-Authored-By: Itamar Hartstein <haritamar@gmail.com>
1 parent 5a4a5d2 commit d398702

2 files changed

Lines changed: 2 additions & 1 deletion

File tree

.github/workflows/test-warehouse.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -192,7 +192,7 @@ jobs:
192192
working-directory: ${{ env.E2E_DBT_PROJECT_DIR }}
193193
continue-on-error: true
194194
run: |
195-
dbt test --target "${{ inputs.warehouse-type }}" --vars '{disable_dbt_artifacts_autoupload: true}'
195+
dbt test --target "${{ inputs.warehouse-type }}"
196196
197197
- name: Run help
198198
run: edr --help

tests/e2e_dbt_project/dbt_project.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ vars:
2121
debug_logs: "{{ env_var('DBT_EDR_DEBUG', False) }}"
2222
custom_run_started_at: "{{ modules.datetime.datetime.utcfromtimestamp(0) }}"
2323
clean_elementary_temp_tables: false
24+
disable_dbt_artifacts_autoupload: true
2425

2526
seeds:
2627
+schema: test_seeds

0 commit comments

Comments
 (0)