Future of removeTrailingWhitespace

quchen / prettyprinter

A modern, extensible and well-documented prettyprinter.

BSD 2-Clause "Simplified" License

294 stars 36 forks source link

Future of removeTrailingWhitespace #170

Open sjakobi opened 4 years ago

sjakobi commented 4 years ago

With https://github.com/quchen/prettyprinter/pull/139 merged, rTW is no longer necessary for the scenarios for which it was originally created. Should we deprecate and remove it now?

sjakobi commented 4 years ago

A few thoughts:

rTW is still useful as a one-stop shop for addressing trailing whitespace
We should at least document that rTW is less useful than it used to be

quchen commented 2 years ago

See also the regression discovered in #205 on why post-precessing trailing whitespace isn’t such a bad idea ;-)

I’ve seen the trailing whitespace cycle a couple of times now – it’s fixed, ah wait no, but now it’s fixed, ah wait no. It’s one of those problems that sound simple but totally aren’t :-(