-
Notifications
You must be signed in to change notification settings - Fork 9
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
Bug: properties gets argument in a strange form with a curvy brace } at the end and fails #3
Comments
Thanks for reporting this @Gregory108. I'm looking into it. At first glance, I'm thinking the issue may stem from us not correctly handling Windows paths. I'll update here once I have more information. |
Hi, Having the same problem. I found the problem at least but not sure of the solution: dbt-invoke properties generates this: when it should be this:
So for some reason the script generates those Setup:
|
@jcopr thanks for posting that information! I also, simultaneously to you, narrowed it down to the quoting of the args. I see that you found the Pull Request #4 which is now merged. A new release has been created (dbt-invoke==0.1.1). Are you (or @Gregory108) able to update to the latest version of dbt-invoke and confirm whether the changes alleviate this issue for you? |
I'll test tomorrow, will comment back here then. And thanks @robastel this seems like an excellent package, we have been using dbt now for 3 years or so 👍 |
@jcopr I'm just checking in to see if you had a chance to test. If you are busy, no worries, I'll close for now and reopen the issue if it pops up again. And thanks for the compliment! |
@robastel It works! Thank you so much! |
@Gregory108 Excellent, I'm glad to hear about the workflow! And thanks for confirming that it's working for you now. |
Execution of:
dbt-invoke properties --models model_name
Gives error:
No idea, how to debug it!)
Setup:
The text was updated successfully, but these errors were encountered: