Skip to content
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

Revisit unicode-to-unicode confusables in RUF001, RUF002, RUF003 #14433

Open
dylwil3 opened this issue Nov 18, 2024 · 0 comments
Open

Revisit unicode-to-unicode confusables in RUF001, RUF002, RUF003 #14433

dylwil3 opened this issue Nov 18, 2024 · 0 comments
Labels
preview Related to preview mode features rule Implementing or modifying a lint rule

Comments

@dylwil3
Copy link
Collaborator

dylwil3 commented Nov 18, 2024

In #8473 the RUF rules regarding confusable/ambiguous unicode were extended, in preview, to include cases where one might confuse some non-ASCII unicode character with another (as opposed to only treating cases where a non-ASCII unicode character may be confused for an ASCII character).

However, the determination of what counts as "confusable", or which of a confusable pair may be preferred, needs some re-examination. The main example encountered in an ecosystem check is the Greek letter μ, which may be used as a variable name (in which case one wants the Greek letter U+03BC) or as part of an SI unit like µs (in which case one wants the "micro" symbol U+00B5).

See the comments on #14403 .

@dylwil3 dylwil3 added rule Implementing or modifying a lint rule preview Related to preview mode features labels Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
preview Related to preview mode features rule Implementing or modifying a lint rule
Projects
None yet
Development

No branches or pull requests

1 participant