Skip to content

fix: add default (empty) tags to dbt builtin globals config#5506

Merged
newtonapple merged 1 commit intomainfrom
ddai/fix-jinja-globals-tags-config
Oct 8, 2025
Merged

fix: add default (empty) tags to dbt builtin globals config#5506
newtonapple merged 1 commit intomainfrom
ddai/fix-jinja-globals-tags-config

Conversation

@newtonapple
Copy link
Contributor

@newtonapple newtonapple commented Oct 8, 2025

There are some DBT dependencies like dbt-unit-testing (<= v0.3.0) that would read from config.tags and expect it to be a Sequence. Without this default, the DBT projects that have dependencies like dbt-unit-testing <= v0.3.0 will fail to load with errors like this:

Error: 'source' macro failed for '<relation>' with exeception 'argument of type 'NoneType' is not iterable'.

@newtonapple newtonapple force-pushed the ddai/fix-jinja-globals-tags-config branch from 547bc72 to 90ce6f8 Compare October 8, 2025 18:06
@newtonapple newtonapple changed the title fix: add default tags config to DbtContext.jinja_globals fix: add default (empty) tags to dbt builtin globals config Oct 8, 2025
@newtonapple newtonapple force-pushed the ddai/fix-jinja-globals-tags-config branch from 90ce6f8 to a7cfd04 Compare October 8, 2025 20:05
@newtonapple newtonapple enabled auto-merge (squash) October 8, 2025 20:27
@newtonapple newtonapple merged commit 189208e into main Oct 8, 2025
36 checks passed
@newtonapple newtonapple deleted the ddai/fix-jinja-globals-tags-config branch October 8, 2025 20:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants