[clang-tools-extra] issue-63565: community requested small QoL fix for more configurabili… (PR #108005)

Nathan Ridge via cfe-commits cfe-commits at lists.llvm.org
Sun Sep 22 00:58:45 PDT 2024


HighCommander4 wrote:

> > but it confuses things when "git blame" says that the last commit that touched some unrelated....
> 
> And it's not confusing if "git blame" says that this same part of code is there in a "clang-format commit". Instead of it's original?

I would say that's less confusing, because you know you can just skip that sort of commit in the blame.

I'm not trying to be difficult here; "please remove unrelated formatting changes" is something I've been told by other reviewers in response to my own patches.

https://github.com/llvm/llvm-project/pull/108005


More information about the cfe-commits mailing list