Disable static tidy check

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.

In the future if we find this check useful, we can turn it on.

Change-Id: Icaa55054e2c51d4ba4ea8f83c2486262deb877c8
Signed-off-by: Jagpal Singh Gill <paligill@gmail.com>
diff --git a/.clang-tidy b/.clang-tidy
index 15ad58f..13342a3 100644
--- a/.clang-tidy
+++ b/.clang-tidy
@@ -253,7 +253,6 @@
 readability-redundant-string-cstr,
 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,