N814 false positive: from decimal import Decimal as D

[rdaysky]

N814 (camelcase-imported-as-constant) triggers even if the alias is a single uppercase letter in code like

from decimal import Decimal as D
from fractions import Fraction as F

but perhaps it shouldn’t?

[charliermarsh]

I tend to agree that we shouldn't flag this.

[rdaysky]

It’s also an interesting question whether from collections.abc import AsyncGenerator as AG is acceptable style for a module that defines a lot of them.

[dhruvmanila]

Did you mean N817 instead of N814? These won't be flagged by the latter as it'll be considered as an acronym for the imported name. See https://play.ruff.rs/9326f4aa-a8bf-47c0-a45f-5b3d04b902c7.

[dhruvmanila]

You can also use the extend-aliases option here:

[tool.ruff.lint.flake8-import-conventions.extend-aliases]
"collections.abc.AsyncGenerator" = "AG"
"decimal.Decimal" = "D"
"fractions.Fraction" = "F"

[dhruvmanila]

I think I'd suggest to use the configuration instead of adding special casing for certain imports. What do you think? @rdaysky