Skip to content

Commit

Permalink
test-all-warehouses: limit matrix on pull requests (#552)
Browse files Browse the repository at this point in the history
* test-all-warehouses: limit matrix on pull requests

* test-all-warehouses: attempt fix

* test-all-warehouses: bugfix to "include" section
  • Loading branch information
haritamar authored Sep 26, 2023
1 parent aee480a commit dd492e0
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion .github/workflows/test-all-warehouses.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,10 @@ jobs:
strategy:
fail-fast: false
matrix:
dbt-version: ${{ inputs.dbt-version && fromJSON(format('["{0}"]', inputs.dbt-version)) || fromJSON('["1.3.0", "latest_official"]') }}
dbt-version:
${{ inputs.dbt-version && fromJSON(format('["{0}"]', inputs.dbt-version)) ||
! contains(github.event_name, 'pull_request') && fromJSON('["1.3.0", "latest_official"]') ||
fromJSON('["latest_official"]') }}
warehouse-type:
[
postgres,
Expand All @@ -48,6 +51,10 @@ jobs:
databricks_catalog,
spark,
]
include:
# If we're not running on a specific dbt version, then always add postgres on 1.3.0
- dbt-version: "${{ inputs.dbt-version || '1.3.0' }}"
warehouse-type: postgres
uses: ./.github/workflows/test-warehouse.yml
with:
warehouse-type: ${{ matrix.warehouse-type }}
Expand Down

0 comments on commit dd492e0

Please sign in to comment.