Zanie Blue
Zanie Blue
Sounds good to me!
To clarify, this looks like it stabilizes the preview behavior introduced in https://github.com/astral-sh/ruff/pull/9676 correct?
Thanks this makes sense to me but we need to wait until 0.5.0 to merge.
Seems reasonable to change the exit code if we fail to parse any files — I'm unsure of the difficulty to implement this.
What's the two line change? :) I think we want to still lint the remaining files, so only exit at the end.
I don't think exiting on the first error is an acceptable solution, unfortunately. We have people that rely on the current behavior and we can perform some lints despite syntax...
I believe @dhruvmanila might eventually be looking into how we can surface parse errors as diagnostics?
Just wondering, what happens if a class has two `Generic[]` bases?
> Is it okay to add the autofix in the same PR? Yep!
Good to know. We should make sure the rule doesn't behave poorly on that case regardless, i.e. it could cause an infinite fix loop.