Skip to content

Fix: properly handle empty vars mapping in dbt_project.yml#5248

Merged
georgesittas merged 1 commit intomainfrom
jo/fix_empty_vars_dbt
Aug 28, 2025
Merged

Fix: properly handle empty vars mapping in dbt_project.yml#5248
georgesittas merged 1 commit intomainfrom
jo/fix_empty_vars_dbt

Conversation

@georgesittas
Copy link
Contributor

The added test fails without this fix:

>       all_project_variables = {**project_yaml.get("vars", {}), **(variable_overrides or {})}
E       TypeError: 'NoneType' object is not a mapping

@georgesittas georgesittas requested a review from a team August 28, 2025 13:16
@georgesittas georgesittas force-pushed the jo/fix_empty_vars_dbt branch from 81c67ed to 5d98263 Compare August 28, 2025 13:21
@georgesittas georgesittas force-pushed the jo/fix_empty_vars_dbt branch from 5d98263 to c8e58f7 Compare August 28, 2025 14:40
@georgesittas georgesittas merged commit 5318b81 into main Aug 28, 2025
35 of 36 checks passed
@georgesittas georgesittas deleted the jo/fix_empty_vars_dbt branch August 28, 2025 16:55
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