Zanie Blue
Zanie Blue
Hi, please just upvote the original post with 👍 if you want to see this. Otherwise we'll report back with any updates and would appreciate if the thread was focused...
I'd be down to have all the formatting run in a single job, yeah
It's possible we missed this in the `0.2.0` stabilizations. For what it's worth, I think this is a bug and we should fix it immediately.
We'd appreciate contributions here if someone is interested in working on the project. We can do this relatively incrementally, but there is some initial complexity to determine how diagnostics are...
The `rasa` ecosystem checks look like a bunch of false positives due to methods that are overriding an [abstract method which must be async](https://github.com/RasaHQ/rasa/blob/cca30d4e06af5aba177e916d64c60313fc537005/rasa/core/actions/action.py#L241-L263). This common enough that we should...
Here's another interesting edge-case false positive https://github.com/zulip/zulip/blob/35098f49597895718343091881fbd6198bd2022d/zerver/tornado/views.py#L35 — this one I'm less sure we can/should do anything about.
Perhaps @AlexWaygood can elucidate whether or not this should trigger for functions with `yield`. https://peps.python.org/pep-0525 may be helpful.
@kkom we're just cutting scope from the initial rule. We could consider it in the future.
It does seems reasonable to include a rule code for this, but given that we cannot parse the noqa directive it could be troublesome as you won't be able to...
I'm surprised this _never_ occurs in the current ecosystem checks. Is there a project that uses this syntax that we could test on for a real world case?