Improve error logging for tag push

nebula-plugins · Jan 17, 2024 · 4e96a06 · 4e96a06
1 parent 714cd39
commit 4e96a06
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/src/integTest/groovy/nebula/plugin/release/ReleasePluginIntegrationSpec.groovy b/src/integTest/groovy/nebula/plugin/release/ReleasePluginIntegrationSpec.groovy
@@ -1212,7 +1212,7 @@ class ReleasePluginIntegrationSpec extends GitVersioningIntegrationTestKitSpec {
         git.checkout(branch: 'main', createBranch: true)
 
         when:
-        def result = runTasks('final')
+        def result = runTasks('final', '-i')
 
         then:
         result.task(':final').outcome == TaskOutcome.SUCCESS

diff --git a/src/main/groovy/nebula/plugin/release/git/base/ReleaseTask.groovy b/src/main/groovy/nebula/plugin/release/git/base/ReleaseTask.groovy
@@ -31,10 +31,10 @@ abstract class ReleaseTask extends DefaultTask {
         GitWriteCommandsUtil gitCommands = gitWriteCommandsUtil.get()
         String tagName = tagStrategy.get().maybeCreateTag(gitCommands, projectVersion.get())
         if (tagName) {
-            logger.warn('Pushing changes in {} to {}', tagName, remote.get())
+            logger.info('Pushing changes in {} to {}', tagName, remote.get())
             gitCommands.pushTag(remote.get(), tagName)
         } else {
-            logger.warn('No new tags to push for {}', remote.get())
+            logger.info('No new tags to push for {}', remote.get())
         }
     }
 }
diff --git a/src/main/groovy/nebula/plugin/release/git/command/GitWriteCommandsUtil.groovy b/src/main/groovy/nebula/plugin/release/git/command/GitWriteCommandsUtil.groovy
@@ -38,6 +38,9 @@ class GitWriteCommandsUtil implements Serializable {
         try {
             executeGitCommand("push", remote, tag)
         } catch (Exception e) {
+            if(e.message.contains("* [new tag]")){
+                return
+            }
             logger.error("Failed to push tag ${tag} to remote ${remote}", e)
         }
     }