chore(advisor): Prefer also over let when not mapping

It can be confusing to use `let` if the goal is not to map the (anyway discarded) return value, so use `also` instead to emphasize that just some additional work (logging in this case) is done. Signed-off-by: Sebastian Schuberth <[email protected]>
oss-review-toolkit · Dec 5, 2024 · 2e31827 · 2e31827
1 parent b5cc0ea
commit 2e31827
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/advisor/src/main/kotlin/Advisor.kt b/advisor/src/main/kotlin/Advisor.kt
@@ -93,7 +93,7 @@ class Advisor(
                                 "vulnerabilities via ${provider.descriptor.displayName}. "
                         }
 
-                        providerResults.keys.takeIf { it.isNotEmpty() }?.let { pkgs ->
+                        providerResults.keys.takeIf { it.isNotEmpty() }?.also { pkgs ->
                             logger.debug {
                                 "Affected packages:\n\n${pkgs.joinToString("\n") { it.id.toCoordinates() }}\n"
                             }