-
Notifications
You must be signed in to change notification settings - Fork 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
Add flake8-debugger #909
Add flake8-debugger #909
Conversation
a89397c
to
a6c1071
Compare
a6c1071
to
eaf7c27
Compare
@charliermarsh, hi. I tried to bring the logic as close to the original as possible, but in debugger_call it was not completely successful. I will be glad to receive feedback. |
Awesome! Will review this shortly. Thanks for putting this together. |
@karpa4o4 - I did some small refactors to use |
But let me know if I messed anything up please :) |
@charliermarsh, thanks. Sorry, I forgot to write "Closed" in the pull request description. So issue #546 stays open. |
Oh good catch! Just closed the issue.
Definitely! There's a bunch of unclaimed stuff in #827, if you're interested in those? Alternatively, there's #458 but it's a bit more involved as I'd like it to be integrated with some of the existing docstring checks. |
Oh, or #870 -- we only need the first rule (U100). It'd probably look like the implementation we have for |
@charliermarsh, hi. Unfortunately I could not do the issues that you described above. But now I want to participate in the development of ruff again, can you tell me how I can help? |
No worries! If you're interested, you could consider working on the |
No description provided.