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
B020 Bug #262
Comments
Yeah, I agree if we can find a way to see the scope of student in the lambda I'd happy take that PR. On the contrary, I do argue tho, it would do nothing but help readability to not use the variable name for student in sorted(students, key=lambda student_to_sort: student_to_sort.last_name): |
@cooperlees Seems to be fixed now so can be closed 👍 |
Thanks! Do we know the PR or how did you test it to see it pass? |
@cooperlees Just copied the example above and ran flake8 on it. Looks like this issue is actually a duplicate of #256 which was fixed in #259 although was not actually released when this issue was made. With |
Many thanks explaining in case someone disagrees in the future with us for closing this! |
This is similar to #235 but with sort keys.
results in B020 but it seems like it should be fine.
The text was updated successfully, but these errors were encountered: