[clang-tools-extra] [clang-tidy] Contributing.rst update snippet and docs (PR #129209)

2025-02-28 Thread Marco C. via cfe-commits
https://github.com/Marcondiro updated https://github.com/llvm/llvm-project/pull/129209 >From c1bb0f84852e223a83638384188e4858877ec89e Mon Sep 17 00:00:00 2001 From: Marcondiro <46560192+marcond...@users.noreply.github.com> Date: Fri, 28 Feb 2025 09:34:30 +0100 Subject: [PATCH] [clang-tidy] Contr

[clang-tools-extra] [clang-tidy] Contributing.rst update snippet and docs (PR #129209)

2025-02-28 Thread Marco C. via cfe-commits
Marcondiro wrote: @carlosgalvezp I do not have the ability to merge the PR, could you do it? thanks! https://github.com/llvm/llvm-project/pull/129209 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listin

[clang-tools-extra] [clang-tidy] Contributing.rst update snippet and docs (PR #129209)

2025-02-28 Thread Marco C. via cfe-commits
https://github.com/Marcondiro updated https://github.com/llvm/llvm-project/pull/129209 >From 58972d1e22af7306ae3583a487cb9bf466714b58 Mon Sep 17 00:00:00 2001 From: Marcondiro <46560192+marcond...@users.noreply.github.com> Date: Fri, 28 Feb 2025 09:34:30 +0100 Subject: [PATCH] [clang-tidy] Contr

[clang-tools-extra] [clang-tidy] Contributing.rst update snippet and docs (PR #129209)

2025-02-28 Thread Marco C. via cfe-commits
https://github.com/Marcondiro updated https://github.com/llvm/llvm-project/pull/129209 >From fbd2aa34805c3415f66410ae282f0ceeb7bd7b64 Mon Sep 17 00:00:00 2001 From: Marcondiro <46560192+marcond...@users.noreply.github.com> Date: Fri, 28 Feb 2025 09:34:30 +0100 Subject: [PATCH] [clang-tidy] Contr

[clang-tools-extra] [clang-tidy] Contributing.rst update snippet and docs (PR #129209)

2025-02-28 Thread Marco C. via cfe-commits
@@ -203,6 +202,10 @@ for more information) that will find the pattern in the AST that we want to inspect. The results of the matching are passed to the ``check`` method, which can further inspect them and report diagnostics. +By default, the new check applies only to C++ code

[clang-tools-extra] [clang-tidy] Contributing.rst update snippet and docs (PR #129209)

2025-02-28 Thread Marco C. via cfe-commits
https://github.com/Marcondiro updated https://github.com/llvm/llvm-project/pull/129209 >From afe211ecfa254e12fdf448afe992ed1b2a7d Mon Sep 17 00:00:00 2001 From: Marcondiro <46560192+marcond...@users.noreply.github.com> Date: Fri, 28 Feb 2025 09:34:30 +0100 Subject: [PATCH] [clang-tidy] Contr

[clang-tools-extra] [clang-tidy] Contributing.rst update snippet and docs (PR #129209)

2025-02-28 Thread Marco C. via cfe-commits
https://github.com/Marcondiro created https://github.com/llvm/llvm-project/pull/129209 This reflects the add_new_check.py changes: isLanguageVersionSupported is now overridden by default by the script The changes were instroduced in https://github.com/llvm/llvm-project/pull/100129 Thanks >Fr

[clang-tools-extra] Extend support for specifying languages and version in add_new_check.py (PR #100129)

2025-02-26 Thread Marco C. via cfe-commits
Marcondiro wrote: Hello @njames93, After this PR, the class derived from `ClangTidyCheck` created by the script `add_new_check.py` overrides `isLanguageVersionSupported` by default, restricting the check's scope to `LangOpts.CPlusPlus` code. Running `clang-tidy/add_new_check.py readability awe