Disable static tidy checks
This commit disables two checks that are blocking the inclusion of
coroutine reactors into dbus-sensors.
readability-static-accessed-through-instance
This check doesn't play nice with coroutines, and suggests that
sdbusplus::async::task always use some internal variation. This is
obviously wrong, and hopefully is fixed in a newer clang. Disable it
for now. This check was dubious in its mtaintainability requirement,
but did make code more consistent.
cppcoreguidelines-avoid-reference-coroutine-parameters
This "rule" is even debated in an open issue in cpp core guidelines [1].
Disable it for now.
In the future if we find either of these checks useful, we can turn them
back on.
[1] https://github.com/isocpp/CppCoreGuidelines/issues/2050
Change-Id: I93b5fce8937efabf1f3d02a0b4fde3795cb7b45e
Signed-off-by: Ed Tanous <etanous@nvidia.com>
diff --git a/.clang-tidy b/.clang-tidy
index ab81074..e345ab9 100644
--- a/.clang-tidy
+++ b/.clang-tidy
@@ -232,7 +232,6 @@
cppcoreguidelines-avoid-c-arrays,
cppcoreguidelines-avoid-capturing-lambda-coroutines,
cppcoreguidelines-avoid-goto,
-cppcoreguidelines-avoid-reference-coroutine-parameters,
cppcoreguidelines-c-copy-assignment-signature,
cppcoreguidelines-explicit-virtual-functions,
cppcoreguidelines-init-variables,
@@ -359,7 +358,6 @@
readability-reference-to-constructed-temporary,
readability-simplify-boolean-expr,
readability-simplify-subscript-expr,
-readability-static-accessed-through-instance,
readability-static-definition-in-anonymous-namespace,
readability-string-compare,
readability-suspicious-call-argument,