refactor: replace if MatchString with FindStringSubMatch

Signed-off-by: Jesse Simpson <jesse.simpson36@gmail.com>

pkg/engine/engine.go

@@ -382,8 +382,7 @@ func reformatExecErrorMsg(filename string, err error) error {
 		}
 
 		var traceable TraceableError
-		if execErrFmt.MatchString(current.Error()) {
+		if matches := execErrFmt.FindStringSubmatch(current.Error()); matches != nil {
-			matches := execErrFmt.FindStringSubmatch(current.Error())
 			templateName := matches[execErrFmt.SubexpIndex("templateName")]
 			functionName := matches[execErrFmt.SubexpIndex("functionName")]
 			locationName := matches[execErrFmt.SubexpIndex("location")]
@@ -393,8 +392,7 @@ func reformatExecErrorMsg(filename string, err error) error {
 				message:          errMsg,
 				executedFunction: "executing " + functionName + " at " + locationName + ":",
 			}
-		} else if execErrFmtWithoutTemplate.MatchString(current.Error()) {
+		} else if matches := execErrFmtWithoutTemplate.FindStringSubmatch(current.Error()); matches != nil {
-			matches := execErrFmt.FindStringSubmatch(current.Error())
 			templateName := matches[execErrFmt.SubexpIndex("templateName")]
 			errMsg := matches[execErrFmt.SubexpIndex("errMsg")]
 			traceable = TraceableError{