[clang-tools-extra] [clangd] forward clang-tidy's readability-identifier-naming fix to textDocument/rename (PR #78454)

Nathan Ridge via cfe-commits cfe-commits at lists.llvm.org
Sun Feb 11 15:49:01 PST 2024


HighCommander4 wrote:

Something else I noticed while trying out the patch locally: before the patch, the description of the quick fix in the editor is "change 'foo' to 'Foo'", i.e. a description of what the code action will do.

After the patch, the description of the quick fix is now "invalid case style for function 'foo'", i.e. it describes the problem, not the fix.

We should keep the description of the code action the same, i.e. describing the fix.

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


More information about the cfe-commits mailing list