False negative with useless-expression on strings

[NeilGirdhar]

For some reason useless-expression is not triggered by loose strings. Should these be caught? If comments are intended, then Ruff could suggest converting them comments?

class X:
    """Some docstring."""


def f() -> None:
    """Some docstring."""
    [123]  # noqa: B018
    "Some loose string."  # Should be B018.


x = 1
"""Some editors consider this a variable docstring, but it doesn't affect any __doc__ and
ordinary comments are ubiquitous."""

[charliermarsh]

The intention here is to avoid flagging docstrings, but I think we need to make it more targeted.

[NeilGirdhar]

Yeah, that's what I figured. I tried to illustrate some reasonable uses of strings. I'm not sure how you feel about the "variable docstring", which some editors recognize. As far as I know, they're rarely used and not really a Python concept. They look pretty weird in code:

length: int  # The length of the object.
width: int  # The width of the object.

versus

length: int
"The length of the object."
width: int
"The width of the object."

[charliermarsh]

Yeah, I think we do need to ignore attribute docstrings. They're not standardized, but they're popular enough that I think it'd be a net-negative to flag them.

[dhruvmanila]

I think this can be fixed because our docstring detection is quite accurate now.