diff --git a/.clang-tidy b/.clang-tidy
index 775d2d3..17375d0 100644
--- a/.clang-tidy
+++ b/.clang-tidy
@@ -260,6 +260,7 @@
 readability-deleted-default,
 readability-function-size,
 readability-identifier-naming,
+readability-implicit-bool-conversion,
 readability-isolate-declaration,
 readability-make-member-function-const,
 readability-misleading-indentation,
diff --git a/manager/lamptest/lamptest.cpp b/manager/lamptest/lamptest.cpp
index f91a085..f7f9e90 100644
--- a/manager/lamptest/lamptest.cpp
+++ b/manager/lamptest/lamptest.cpp
@@ -246,7 +246,7 @@
 void LampTest::timeOutHandler()
 {
     // set the Asserted property of lamp test to false
-    if (!groupObj)
+    if (groupObj == nullptr)
     {
         lg2::error("the Group object is nullptr");
         throw std::runtime_error("the Group object is nullptr");
diff --git a/manager/manager.cpp b/manager/manager.cpp
index 3dfa92a..df65426 100644
--- a/manager/manager.cpp
+++ b/manager/manager.cpp
@@ -273,7 +273,7 @@
         lg2::debug("De-Asserting LED, NAME = {NAME}, ACTION = {ACTION}", "NAME",
                    it.name, "ACTION", it.action);
         if (drivePhysicalLED(objPath, Layout::Action::Off, it.dutyOn,
-                             it.period))
+                             it.period) != 0)
         {
             failedLedsDeAssert.insert(it);
         }
@@ -284,7 +284,7 @@
         std::string objPath = std::string(phyLedPath) + it.name;
         lg2::debug("Asserting LED, NAME = {NAME}, ACTION = {ACTION}", "NAME",
                    it.name, "ACTION", it.action);
-        if (drivePhysicalLED(objPath, it.action, it.dutyOn, it.period))
+        if (drivePhysicalLED(objPath, it.action, it.dutyOn, it.period) != 0)
         {
             failedLedsAssert.insert(it);
         }