-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
[CT-1591] Don't parse empty Python files #6433
Conversation
I've confirmed the following, I think PR should be good to merge?
P.S. I misclicked on closing this PR, hence the noise below. Footnotes
|
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.
Looks great!! Thanks for testing out the behavior of run, list and doc
With this change now being live should we not be seeing this behavior anymore in dbt cloud? I just created a new .py file and this threw the same problem. I was the original poster/opened the bug with the corresponding loom so I am not resposting here in the comment. |
Resolves #6367
Description
Only parse Python AST if body is not empty
Checklist
changie new
to create a changelog entry