#43 Refactoring: Use logger in mojo

- instead of println
georgberky · Oct 5, 2022 · 98343cd · 98343cd
1 parent cee8bb1
commit 98343cd
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/src/main/kotlin/io/github/georgberky/maven/plugins/depsupdate/UpdateMojo.kt b/src/main/kotlin/io/github/georgberky/maven/plugins/depsupdate/UpdateMojo.kt
@@ -44,13 +44,13 @@ class UpdateMojo : AbstractMojo() {
                     artifactFactory = artifactFactory
             )
             .updates
-            .onEach { println("execute im mojo: latestVersion: '${it.latestVersion}' / version:'${it.version}'") }
+            .onEach { log.debug("execute in mojo: latestVersion: '${it.latestVersion}' / version:'${it.version}'") }
             .filterNot (VersionUpdate::isAlreadyLatestVersion)
-            .onEach { println("execute im mojo canSkip: ${it.latestVersion}") }
+            .onEach { log.debug("execute in mojo after latest version check: ${it.latestVersion}") }
             .map { it to "dependency-update/${it.groupId}-${it.artifactId}-${it.latestVersion}" }
-            .onEach { println("execute im mojo canSkip (nach map): ${it.second} ${it.first}") }
+            .onEach { log.debug("execute in mojo canSkip (after map): ${it.second} ${it.first}") }
             .filter { (_, branchName) -> !git.hasRemoteBranch(branchName) }
-            .onEach { println("execute im mojo nach filter branches: ${it.second} ${it.first}") }
+            .onEach { log.debug("execute in mojo after filter branches: ${it.second} ${it.first}") }
             .forEach { (update, branchName) ->
                 git.checkoutNewBranch(branchName)
                 val pom = update.updatedPom()

diff --git a/src/main/kotlin/io/github/georgberky/maven/plugins/depsupdate/VersionUpdate.kt b/src/main/kotlin/io/github/georgberky/maven/plugins/depsupdate/VersionUpdate.kt
@@ -11,6 +11,7 @@ abstract class VersionUpdate(
     abstract fun updatedPom() : Document
 
     fun isAlreadyLatestVersion() : Boolean {
+        // TODO #12: use DI-injected logger
         println("canSkip: ${latestVersion} compare to ${version}")
         println("latestVersion.equals(version) = ${latestVersion.equals(version)}")
         return latestVersion != "null" && latestVersion != version