-
Notifications
You must be signed in to change notification settings - Fork 10
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
Added support for multiple fixes per block in fix-file-resouce-leak #245
Conversation
) | ||
# is closed? | ||
name_condition = map( | ||
# pylint: disable-next=cell-var-from-loop |
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.
hmmm either this pylint warning is worth listening to or it's a pylint bug? Either way, maybe a list-comp is easier to read here?
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.
I wanted to avoid building the list since the result would just be fed into all
. I don't exactly remember the issue with the warning was. It may be invalid by now. I'll try to remove and see if it still complains.
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.
Personal preference nit pick would be to use less one-letter variable names as it's harder to grep for them .
4002e13
to
f549be1
Compare
file_context: FileContext, | ||
*codemod_args, | ||
*codemod_args, # pylint: disable=unused-argument |
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.
Small thing but I think the idiomatic way to suppress this is with
del *codemod_args
Although possibly this argument is no longer needed?
f549be1
to
f380d9a
Compare
Quality Gate passedIssues Measures |
Overview
fix-file-resource-leak
will now fix multiple leaks in a single block at once