clang-tidy: Enable performance-unnecessary-value-param check

This check flags value parameter declarations of expensive to copy
types that are copied for each invocation but it would suffice to
pass them by const reference.

Change-Id: I631deda63fcbb74362313c9596bf7e72933b0d0c
Signed-off-by: Pavithra Barithaya <pavithrabarithaya07@gmail.com>
diff --git a/image_manager.cpp b/image_manager.cpp
index 8e32e3f..c9a958d 100644
--- a/image_manager.cpp
+++ b/image_manager.cpp
@@ -240,7 +240,7 @@
     return 0;
 }
 
-void Manager::erase(std::string entryId)
+void Manager::erase(const std::string& entryId)
 {
     auto it = versions.find(entryId);
     if (it == versions.end())