-
Notifications
You must be signed in to change notification settings - Fork 16.3k
Update prek hook to check for session.query() #60510
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
Conversation
|
nice! |
(cherry picked from commit 62f2804) Co-authored-by: Pratiksha <128999446+Prab-27@users.noreply.github.com>
|
There is also a looong list in pre-merge checks for |
Yes, I’ll do that once we’ve cleaned up the entire module. I’ll update it then. We’re almost done here — just a few tasks left, and after that we can customize this. Here’s the TODO: #47275 (comment) . I’d love to hear from you !! |
Cool! No all fine. I am not an expert but a noob in this. I am grateful that you are leading this! |
It helps me a lot! I really appreciate this idea!! |
Related discussion : here and here
From now on, this also checks for the entire
session.query()usage, so we can remove it from the remaining providersWas generative AI tooling used to co-author this PR?
{pr_number}.significant.rstor{issue_number}.significant.rst, in airflow-core/newsfragments.