From 65084371c9a774c6c4c6afadd5f6fa2152b31b8a Mon Sep 17 00:00:00 2001 From: Jesse Simpson Date: Mon, 17 Mar 2025 12:39:55 -0400 Subject: [PATCH] refactor: replace if MatchString with FindStringSubMatch Signed-off-by: Jesse Simpson --- pkg/engine/engine.go | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/pkg/engine/engine.go b/pkg/engine/engine.go index ad2c1679d..26f80a2f0 100644 --- a/pkg/engine/engine.go +++ b/pkg/engine/engine.go @@ -382,8 +382,7 @@ func reformatExecErrorMsg(filename string, err error) error { } var traceable TraceableError - if execErrFmt.MatchString(current.Error()) { - matches := execErrFmt.FindStringSubmatch(current.Error()) + if matches := execErrFmt.FindStringSubmatch(current.Error()); matches != nil { 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()) { - matches := execErrFmt.FindStringSubmatch(current.Error()) + } else if matches := execErrFmtWithoutTemplate.FindStringSubmatch(current.Error()); matches != nil { templateName := matches[execErrFmt.SubexpIndex("templateName")] errMsg := matches[execErrFmt.SubexpIndex("errMsg")] traceable = TraceableError{