Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
(same basic story as fivetran/dbt_hubspot_source#46)
Are you a current Fivetran customer?
Jeremy Cohen, dbt Labs, yes!
What change(s) does this PR introduce?
fivetran_utils: [">=0.2.0", "<0.3.0"]
. Lower + upper since you have full control over this package, and can release it whenever you need.dbt >= 0.20.0
. Just a lower bound because we can rely ondbt_utils
(viafivetran_utils
) to hold the upper dbt bound for us.version
(cosmetic only)dbt_project.yml
— this will just use the dbt version requirement in the packages it depends onDoes this PR introduce a breaking change?
Is this PR in response to a previously created Issue
How did you test the PR changes?
Select which warehouse(s) were used to test the PR
I ran this on Postgres, everything passed, with one tiny change: I needed to wrap quotes around the
default
column inbrand_data
, sincedefault
is a reserved word in Postgres.As before, happy to run against the other warehouses if you'd like. With an update like this, if the package parses/compiles/runs right on one, it ought to across the board.
Provide an emoji that best describes your current mood
🕴️ 🕴️