prettyprinter icon indicating copy to clipboard operation
prettyprinter copied to clipboard

Published 20 hours ago • verified publisher icon quchen

Future of removeTrailingWhitespace

Open sjakobi opened this issue 4 years ago • 2 comments

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 avatar Jun 30 '20 01:06 sjakobi

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

sjakobi avatar Jun 30 '20 01:06 sjakobi

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 :-(

quchen avatar Dec 03 '21 13:12 quchen