-
Notifications
You must be signed in to change notification settings - Fork 121
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
Official v1 upgrade #119
Official v1 upgrade #119
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jtcohen6 thanks for dropping the dep on dbt-utils! huge win to decouple them for sure especially for what was such a small dependency.
4e95f8e
to
5e0f22a
Compare
{% do dbt_utils.log_info(loop_label ~ ' SKIP') if run_queue == [] %} | ||
{% set width = 80 %} {# hard code this for now, use PRINTER_WIDTH flag in v1.0+ #} | ||
{% do log(loop_label ~ ' SKIP', info = true) if run_queue == [] %} | ||
{% set width = flags.PRINTER_WIDTH %} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is still causing bugs - why did you remove the hardcoding patch from v0.7.3?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@timbrownsf This version of the package is only for dbt version 1.0 and higher, and the issue was only for versions prior to 1.0 - are you using --no-version-check
in your dbt command? you may want to switch back to 0.7.3 instead.
Description & motivation
dbt_project.yml
to remove deprecation warningsdbt_utils
dependency — this was only being used for logging niceties, which are redundant thanks to new logging in v1Checklist
I have verified that these changes work locallyI have updated the README.md (if applicable)I have added an integration test for my fix/feature (if applicable)