-
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 --target-path to snapshot command #7419
Add --target-path to snapshot command #7419
Conversation
Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA. In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, please reach out through a comment on this PR. CLA has not been signed by users: @dwreeves |
Woops. Just signed the CLA. |
@cla-bot check |
The cla-bot has been summoned, and re-checked this pull request! |
@dbeatty10 Thanks for getting to this. Do you think this can be available in time for the dbt 1.5.0 release? 😅 |
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!
@dwreeves Thanks to your early testing and contribution, this is on track to being included in the 1.5.0 release 🏆 |
resolves #7418
Description
dbt snapshot
uses the target path (since it runs a subclass ofRunTask
), but does not contain the option to specify it. This PR adds--target-path
as a valid option fordbt snaphot
.Checklist
changie new
to create a changelog entry