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

Deprecated: setting log-path + target-path in dbt_project.yml #3014

Closed
1 task done
jtcohen6 opened this issue Mar 17, 2023 · 2 comments · Fixed by #3129
Closed
1 task done

Deprecated: setting log-path + target-path in dbt_project.yml #3014

jtcohen6 opened this issue Mar 17, 2023 · 2 comments · Fixed by #3129
Labels
content Improvements or additions to content dbt-core v1.5 Docs impact for the v1.5 release (Apr 2023) improvement Use this when an area of the docs needs improvement as it's currently unclear

Comments

@jtcohen6
Copy link
Collaborator

Contributions

  • I have read the contribution docs, and understand what's expected of me.

Link to the page on docs.getdbt.com requiring updates

https://docs.getdbt.com/reference/project-configs/log-path
https://docs.getdbt.com/reference/project-configs/target-path

What part(s) of the page would you like to see updated?

We are deprecating the ability to set these configs in dbt_project.yml, in favor of setting them as CLI flags or env vars. This is much more consistent with how other flags are set/passed during app initialization.

In v1.5, there will be backwards compatibility for continuing to set these configs in dbt_project.yml. We plan to remove this back compat in a future (as-of-yet-undetermined) version of dbt-core.

Additional information

dbt-labs/dbt-core#7185

@jtcohen6 jtcohen6 added content Improvements or additions to content improvement Use this when an area of the docs needs improvement as it's currently unclear idea Proposes an idea for new content dbt-core v1.5 Docs impact for the v1.5 release (Apr 2023) labels Mar 17, 2023
@matthewshaver matthewshaver linked a pull request Apr 3, 2023 that will close this issue
7 tasks
@runleonarun runleonarun removed the idea Proposes an idea for new content label Mar 13, 2024
@adamcunnington-mlg
Copy link

@jtcohen6 please can you confirm whether this decision was reversed at a later date? I've noticed the docs no longer say these 2 properties are deprecated but according to Google, they did as of February.

Screenshot_20240326-070302

@jtcohen6
Copy link
Collaborator Author

@adamcunnington-mlg These two configs are still deprecated for setting in dbt_project.yml. They are supported to set via CLI flag (--log-path, --target-path) or environment variable.

We did recently reorganize this documentation:

From the updated page (https://docs.getdbt.com/reference/global-configs/about-global-configs):

Screenshot 2024-03-28 at 15 05 09

And the table further down (with my scrappy edit to show both flags more concisely):

Untitled

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content Improvements or additions to content dbt-core v1.5 Docs impact for the v1.5 release (Apr 2023) improvement Use this when an area of the docs needs improvement as it's currently unclear
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants