Implement W291 (trailing whitespace)?

[harupy]

It'd be nice if Ruff could detect trailing whitespace:

(even though black can eliminate them)

[charliermarsh]

I don't know if it's relevant (haven't thought through how we would implement trailing whitespace), but I started implementing Pycodestyle's logical line detection in #1130 -- although I felt at the time that it was a bit too slow to be worthwhile as a default.

[charliermarsh]

Do these checks typically exclude trailing whitespace within strings?

[not-my-profile]

Black preserves trailing whitespace in multiline strings (as it should). I just checked and pycodestyle does trigger W291 for trailing whitespace within multiline strings. To be honest I don't really see the point of this lint.

[charliermarsh]

I think it's a perfectly useful rule, if you don't use an autoformatter.

[not-my-profile]

Haha yes I don't see why you wouldn't.^^

[harupy]

@charliermarsh @not-my-profile Agreed! Let me close this.

[harupy]

Black preserves trailing whitespace in multiline strings (as it should). I just checked and pycodestyle does trigger W291 for trailing whitespace within multiline strings. To be honest I don't really see the point of this lint.

Yes, black does. We can align with black and ignore trailing whitespace in strings (as shown in the attached screenshot).