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

Add --empty flag value to context flags.EMPTY #10317

Closed
1 task done
MichelleArk opened this issue Jun 17, 2024 · 1 comment · Fixed by #10315
Closed
1 task done

Add --empty flag value to context flags.EMPTY #10317

MichelleArk opened this issue Jun 17, 2024 · 1 comment · Fixed by #10315
Assignees
@MichelleArk
Labels
backport 1.8.latest empty Issues related to the --empty CLI flag user docs [docs.getdbt.com] Needs better documentation

Comments

@MichelleArk
Copy link
Contributor

Housekeeping

  • I am a maintainer of dbt-core

Short description

We are missing the --empty flag value from the jinja context flags, making it tricky to overwrite the ref macro in a way that incorporates the value of --empty

Reported in community slack: https://getdbt.slack.com/archives/C2JRRQDTL/p1718298268904749?thread_ts=1718044905.163699&cid=C2JRRQDTL

Acceptance criteria

flags.EMPTY exists in jinja!

Suggested Tests

  1. Add unit tests :)

Impact to Other Teams

N/A

Will backports be required?

1.8

Context

No response
@MichelleArk MichelleArk added the user docs [docs.getdbt.com] Needs better documentation label Jun 17, 2024
@MichelleArk MichelleArk self-assigned this Jun 17, 2024
@MichelleArk MichelleArk mentioned this issue Jun 17, 2024
Merged
5 tasks
@FishtownBuildBot
Copy link
Collaborator

Opened a new issue in dbt-labs/docs.getdbt.com: dbt-labs/docs.getdbt.com#5664

@nataliefiann nataliefiann mentioned this issue Sep 10, 2024
Open
@dbeatty10 dbeatty10 added the empty Issues related to the --empty CLI flag label Sep 24, 2024
@dbeatty10 dbeatty10 changed the title Add --empty flag value to context flags.EMPTY Add --empty flag value to context flags.EMPTY Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MichelleArk MichelleArk

Labels
backport 1.8.latest empty Issues related to the --empty CLI flag user docs [docs.getdbt.com] Needs better documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

add empty to context flags dbt-labs/dbt-core
3 participants
@MichelleArk @dbeatty10 @FishtownBuildBot