Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Future of removeTrailingWhitespace #170

Open
sjakobi opened this issue Jun 30, 2020 · 2 comments
Open

Future of removeTrailingWhitespace #170

sjakobi opened this issue Jun 30, 2020 · 2 comments

Comments

@sjakobi
Copy link
Collaborator

sjakobi commented Jun 30, 2020

With #139 merged, rTW is no longer necessary for the scenarios for which it was originally created. Should we deprecate and remove it now?

@sjakobi
Copy link
Collaborator Author

sjakobi commented Jun 30, 2020

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
Copy link
Owner

quchen commented Dec 3, 2021

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants