From 1a710c3155cd3273b704be16e5e859a00f19fb64 Mon Sep 17 00:00:00 2001
From: Jim Nicholson <thejimnicholson@gmail.com>
Date: Wed, 7 Feb 2024 02:20:54 -0800
Subject: [PATCH] Try this instead

---
 vars/buildStep.groovy      | 2 +-
 vars/checkoutStep.groovy   | 2 +-
 vars/makeStepBanner.groovy | 5 ++---
 3 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/vars/buildStep.groovy b/vars/buildStep.groovy
index 4acf4e8..fe31dd8 100644
--- a/vars/buildStep.groovy
+++ b/vars/buildStep.groovy
@@ -9,7 +9,7 @@ void call(Map config = [:], Closure body) {
     ]
     Map runConfig = defaults + config
     stage(runConfig.stepName) {
-        echo makeStepBanner(runConfig.stepName)
+        makeStepBanner(runConfig.stepName)
         if (runConfig.saveArtifacts) {
             assert runConfig.artifacts : 'Error: saveArtifacts is set to true; must specify artifacts!'
         }
diff --git a/vars/checkoutStep.groovy b/vars/checkoutStep.groovy
index af4630a..8d72f5d 100644
--- a/vars/checkoutStep.groovy
+++ b/vars/checkoutStep.groovy
@@ -8,7 +8,7 @@ void call(Map config = [:], Closure body) {
     echo 'Firing ...'
     runConfig = defaults + config
     stage(runConfig.stepName) {
-        echo makeStepBanner(runConfig.stepName)
+        makeStepBanner(runConfig.stepName)
         if (runConfig.useJobSCM) {
             checkout scm
         }
diff --git a/vars/makeStepBanner.groovy b/vars/makeStepBanner.groovy
index 2e8dd4d..8a72816 100644
--- a/vars/makeStepBanner.groovy
+++ b/vars/makeStepBanner.groovy
@@ -1,10 +1,9 @@
-String makeStepBanner(String msg) {
-    banner = """
+void makeStepBanner(String msg) {
+    echo """
        ===========================================
 
        ${msg}
 
        ===========================================
        """
-    return banner
 }