We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
On ruff 0.2.0, the following command:
⊙ ruff --version && ruff --select PIE --isolated <(printf ' julian@Airm from typing import Protocol, TypeVar D = TypeVar("D") class Foo(Protocol[D]): """ Foo bar. """ ... ')
produces:
ruff 0.2.0 /dev/fd/11:12:5: PIE790 [*] Unnecessary `...` literal Found 1 error. [*] 1 fixable with the `--fix` option.
but the ellipsis should not be removed.
This is essentially the same issue as #8756 but it looks like the fix put in there only handled non-generic Protocols.
The text was updated successfully, but these errors were encountered:
Ah thanks.
Sorry, something went wrong.
I'll fix this now and it can go out in today's release.
You sir are fast :) Much appreciation as usual!
Protocol
No prob :)
Respect generic Protocol in ellipsis removal (#9841)
041ce1e
Closes #9840.
charliermarsh
Successfully merging a pull request may close this issue.
On ruff 0.2.0, the following command:
produces:
but the ellipsis should not be removed.
This is essentially the same issue as #8756 but it looks like the fix put in there only handled non-generic Protocols.
The text was updated successfully, but these errors were encountered: