Enable more clang-tidy checks and treat them as errors

Test: tests/runtests.sh
Change-Id: If59480cee6460847f5c1cef17e3ef036b8e75651
diff --git a/server/TrafficController.cpp b/server/TrafficController.cpp
index d599a82..b55e667 100644
--- a/server/TrafficController.cpp
+++ b/server/TrafficController.cpp
@@ -59,7 +59,6 @@
 
 using base::StringPrintf;
 using base::unique_fd;
-using base::Join;
 using netdutils::extract;
 using netdutils::Slice;
 using netdutils::sSyscalls;
@@ -721,6 +720,7 @@
     return StringPrintf("OK");
 }
 
+// NOLINTNEXTLINE(google-runtime-references): grandfathered pass by non-const reference
 void dumpBpfMap(const std::string& mapName, DumpWriter& dw, const std::string& header) {
     dw.blankline();
     dw.println("%s:", mapName.c_str());