kadircet wrote: (sorry for the late reply)
> Just want to confirm that this infrastructure isn't currently being used by > anything in Clangd. we do have some out-of-tree usages, but nothing in the tree, apart from tests, use this infrastructure yet. > I assume I should extend FeatureModule to add my NOLINT fix feature and add > it in ClangMain.cpp conditioning on command line options. Would that be the > correct approach? yes, we basically should have a new feature-module, from that new feature-module we can contribute a tweak. this tweak can look at all the diagnostics in the AST, find the ones that intersect with users selection, and if they're from clang-tidy emit a code-action that inserts `` // NOLINTNEXTLINE(check-name)`. https://github.com/llvm/llvm-project/pull/114661 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
