Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

Add DbtCleanOperator #47

Merged
merged 2 commits into from
Oct 25, 2021
Merged

Add DbtCleanOperator #47

merged 2 commits into from
Oct 25, 2021

Conversation

fenimore
Copy link
Contributor

@fenimore fenimore commented Oct 22, 2021 •
edited
Loading

Adding another Operator for the clean command 😇

@dinigo
Copy link

dinigo commented Oct 22, 2021

LGTM. I've been working in the #45 . Feel free to code-review it :)

andrewrjones
andrewrjones approved these changes Oct 25, 2021
View reviewed changes
Copy link
Contributor

@andrewrjones andrewrjones left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @fenimore! LGTM!

@andrewrjones andrewrjones merged commit 6b8af6e into gocardless:master Oct 25, 2021
@fenimore fenimore deleted the dbt-clean-operator branch October 25, 2021 15:56
@fenimore
Copy link
Contributor Author

Thanks for the quick response! Is there a specific cadence for the releases? Or will this automatically be released to pypi?

@andrewrjones
Copy link
Contributor

I just want to review the other open PRs (in particular #48) and then release them all together. So should be in the next couple of days.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@andrewrjones andrewrjones andrewrjones approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@fenimore @dinigo @andrewrjones