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.

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

Rethink rendering of fields in dbt_project.yml and profiles.yml #960

Closed
cmcarthur opened this issue Aug 29, 2018 · 2 comments
Closed

Rethink rendering of fields in dbt_project.yml and profiles.yml #960

cmcarthur opened this issue Aug 29, 2018 · 2 comments
Assignees

Comments

@cmcarthur
Copy link
Member

Feature

Feature description

dbt is not very explicit about which fields in config files support jinja, and which jinja functions they support. We can do a better job of breaking this up into groups, for example:

  • essentially all fields in dbt_project.yml and profiles.yml should support {{env_var(..)}} and {{var(..)}} and should be rendered immediately
  • specific fields in dbt_project.yml support a larger context, for example on-run-end accepts a fuller context

Who will this benefit?

everyone

@cmcarthur
Copy link
Member Author

depends on #953

@beckjake
Copy link
Contributor

implementation note: rendering of all hooks really has to wait for runtime. I think all other fields can and should be rendered immediately, though some (like models/seeds/archive) will be tricky since they're pretty free-form.

beckjake added a commit that referenced this issue Oct 8, 2018
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

No branches or pull requests

2 participants