-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Add logging to dbt clean
#1383
Add logging to dbt clean
#1383
Conversation
@drewbanin I'd love to get your thoughts on this as is (or clarity on what else it needs). |
This looks good to me. Can you rebase off the existing |
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.
lgtm, thank you for updating the contributing docs!
@beckjake should be good to kickoff now! |
@emilieschario I've reopened this PR with your commits against |
Thanks @cmcarthur!! |
This MR does two things:
It adds some small updates to the contributing docs to make it easier for someone to get started without getting lost in the documentation of other tools.
It adds logging to the
dbt clean
task as outlined in Feature: Command line output after dbt clean #1261.A sample output on 0.13.0 might look like