Allow to exclude file types from Files: Trim Trailing Whitespace

[ermshiperete]

I'd like to define exceptions for files.trimTrailingWhitespace so that it doesn't trim trailing whitespace for some file types.

My use case is that I have .patch files. These use trailing tabs and whitespace to detect filenames and end of patch. If I edit and save a .patch file patch is no longer able to apply the patch. I'd like to define an exception so that trimming trailing whitespace doesn't happen for files ending in .patch or of type diff.

[gjsjohnmurray]

Can you handle it this way?

#120468 (comment)

[ermshiperete]

Yes, thanks, adding these lines to settings.json work:

   "[diff]": {
     "files.trimTrailingWhitespace": false
    },

However, that is not at all intuitive in the visual settings editor:

• it is not clear that by searching for "@lang:diff:" you'll see settings that apply only to that language. Searching for other things, even with the @... syntax, only filters what gets displayed, whereas with @lang:... you see a complete independent set of settings.
• if you search for "@lang:diff trimTrailingWhitespace" (without having the setting in settings.json) it looks like the option is already off, even though it will be on because it's set in the global setting.

[rzhao271]

The second issue is #58038
Feel free to recommend/suggest additional designs for the first issue.

[vscodenpa]

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.

If you wonder what we are up to, please see our roadmap and issue reporting guidelines.

Thanks for your understanding, and happy coding!