Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Support vars with null/none values #608

Closed
drewbanin opened this issue Nov 30, 2017 · 0 comments · Fixed by #1426
Closed

Support vars with null/none values #608

drewbanin opened this issue Nov 30, 2017 · 0 comments · Fixed by #1426
Assignees
@drewbanin
Labels
bug Something isn't working good_first_issue Straightforward + self-contained changes, good for new contributors!
Milestone
Wilt Chamberlain

Comments

@drewbanin
Copy link
Contributor

dbt thinks that variables with null values are not set. We should distinguish between

  1. not set
  2. set to none
@drewbanin drewbanin added the bug Something isn't working label Nov 30, 2017
@drewbanin drewbanin added this to the 0.9.1 milestone Nov 30, 2017
@drewbanin drewbanin self-assigned this Nov 30, 2017
@drewbanin drewbanin removed this from the 0.9.1 milestone Dec 14, 2017
@drewbanin drewbanin added the good_first_issue Straightforward + self-contained changes, good for new contributors! label Mar 12, 2018
@drewbanin drewbanin added this to the Wilt Chamberlain milestone Jan 23, 2019
@beckjake beckjake mentioned this issue Apr 29, 2019
Merged
beckjake added a commit that referenced this issue Apr 30, 2019
@beckjake
Merge pull request #1426 from fishtown-analytics/feature/allow-null-vars
4715ad9 
Allow vars to be set to null and differentiate them from unset vars (#608)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@drewbanin drewbanin

Labels
bug Something isn't working good_first_issue Straightforward + self-contained changes, good for new contributors!
Projects
None yet
Milestone
Wilt Chamberlain
Development

Successfully merging a pull request may close this issue.

Allow vars to be set to null and differentiate them from unset vars (#608) dbt-labs/dbt-core
1 participant
@drewbanin