[PATCH] D135789: Add .editorconfig file

Aaron Ballman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 13 11:53:27 PDT 2022


aaron.ballman added a comment.

Do we want to add .editorconfig to .gitignore so users are free to modify their local copy for additional options supported by their editor without accidentally committing the changes for everyone else? Or is that a bad idea?



================
Comment at: .editorconfig:8
+charset = utf-8
+trim_trailing_whitespace = true
----------------
Do these settings apply to the whole file when it is saved, or only to the modifications in the file as they're being made?

I ask because of `trim_trailing_whitespace` specifically -- I have a text editor that does that automatically already when saving a file and I have to configure it to *not* do that because of how often it ends up making unrelated changes in a file when saving; especially our older test cases in Clang.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D135789/new/

https://reviews.llvm.org/D135789



More information about the llvm-commits mailing list