[PATCH] D130863: [clangd] Make git ignore index directories

Tue Aug 2 03:12:11 PDT 2022

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