From 9781011d91db73012e5c179a6f7da75e213393eb Mon Sep 17 00:00:00 2001 From: Max SCHMELLER Date: Wed, 20 Nov 2024 18:06:57 +0900 Subject: [PATCH] chore(clang-tidy): add explanations for disabled checks Signed-off-by: Max SCHMELLER --- .clang-tidy | 14 ++++++++++++++ 1 file changed, 14 insertions(+) diff --git a/.clang-tidy b/.clang-tidy index 0f1ce7e1a..f5cf5ab9a 100644 --- a/.clang-tidy +++ b/.clang-tidy @@ -1,3 +1,17 @@ +# All disabled checks have to come with an explanatory comment that states why they were disabled! +# +# modernize-use-trailing-return-type: +# Purely stylistic, makes function signature longer +# +# readability-identifier-length: +# Things like std::ostream os; nlohmann::json j; are popular. Reviewer can decide on those cases +# +# readability-implicit-bool-conversion: +# Makes packet-parsing (bool field that occupies a byte or bitfield) annoying +# +# hicpp-special-member-functions: +# Alias for cppcoreguidelines-special-member-functions, which is enabled. If both are enabled, +# the CheckOptions below have to be specified twice. Checks: " *, -abseil-*,