-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Fix false negative for calling module-level function before definition #8494
Fix false negative for calling module-level function before definition #8494
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #8494 +/- ##
==========================================
- Coverage 95.81% 95.81% -0.01%
==========================================
Files 174 174
Lines 18321 18319 -2
==========================================
- Hits 17554 17552 -2
Misses 767 767
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This reverts commit befe583.
π€ Effect of this PR on checked open source code: π€ Effect on astroid:
Effect on django:
Effect on poetry-core:
This comment was generated for commit 3fdf7fe |
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 !
I was going to wait for a new primer result, but if there are follow-ups, I'll just open a new PR. Sorry for not saying something sooner! |
That's my bad for merging too fast :) |
No worries, I just checked and I can't reproduce the 2nd and 3rd items from the earlier primer comment, so I think they were also property-related like the 1st one I addressed in astroid (and checking the files confirms it) π |
Type of Changes
Description
Closes #1144