ilya-biryukov added a comment.

I'm with Kadir on this one, but maybe I'm too attached to my 
`.git/info/exclude`s. I somehow don't feel it's a big burden specific to 
clangd. I have to configure `.gitignore` for my projects and 
`.git/info/excludes` for other projects I work on because of many tools I 
personally use (editors, etc).

If this lands, I feel we should say `*` in the added `.gitignore` and put it 
into `.cache/clangd`.
That way the `.cache` folder does not appear in the git status at all and this 
improves the workflow for people who use git, i.e. they don't see anything 
related to clangd's internal storage in their VCS by default.

Otherwise, I would personally still put `.cache/clangd` into the global 
`.gitignore` or `.git/info/exclude`. IMO `.cache/clangd` should not be in the 
VCS history at all, even if only for the `.gitignore` files.
So landing this as is wouldn't really improve the situation for me personally.
Maybe my workflow is different from what others do, but hopefully this can be 
helpful.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130863

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to