From 442200033027ce2512a5974a52993a7af4aac3d6 Mon Sep 17 00:00:00 2001
From: Robert Sirchia <rsirchia@outlook.com>
Date: Wed, 5 Mar 2025 17:21:29 +0100
Subject: [PATCH] fixing case issues with the logging of my errors

Signed-off-by: Robert Sirchia <rsirchia@outlook.com>
---
 pkg/chart/v2/util/dependencies.go | 6 +++---
 pkg/engine/engine.go              | 4 ++--
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/pkg/chart/v2/util/dependencies.go b/pkg/chart/v2/util/dependencies.go
index 80268b45c..493bd31d6 100644
--- a/pkg/chart/v2/util/dependencies.go
+++ b/pkg/chart/v2/util/dependencies.go
@@ -51,7 +51,7 @@ func processDependencyConditions(reqs []*chart.Dependency, cvals Values, cpath s
 					slog.Warn("returned non-bool value", "path", c, "chart", r.Name)
 				} else if _, ok := err.(ErrNoValue); !ok {
 					// this is a real error
-					slog.Error("pathValue returned error", slog.Any("err", err))
+					slog.Error("the method PathValue returned error", slog.Any("err", err))
 				}
 			}
 		}
@@ -254,7 +254,7 @@ func processImportValues(c *chart.Chart, merge bool) error {
 				// get child table
 				vv, err := cvals.Table(r.Name + "." + child)
 				if err != nil {
-					slog.Error("importValues missing table from chart", "chart", r.Name, "value", err)
+					slog.Error("ImportValues missing table from chart", "chart", r.Name, "value", err)
 					continue
 				}
 				// create value map from child to be merged into parent
@@ -271,7 +271,7 @@ func processImportValues(c *chart.Chart, merge bool) error {
 				})
 				vm, err := cvals.Table(r.Name + "." + child)
 				if err != nil {
-					slog.Error("importValues missing table", slog.Any("err", err))
+					slog.Error("ImportValues missing table", slog.Any("err", err))
 					continue
 				}
 				if merge {
diff --git a/pkg/engine/engine.go b/pkg/engine/engine.go
index d47606ee6..9c91fd43b 100644
--- a/pkg/engine/engine.go
+++ b/pkg/engine/engine.go
@@ -176,12 +176,12 @@ func tplFun(parent *template.Template, includedNames map[string]int, strict bool
 		// text string. (Maybe we could use a hash appended to the name?)
 		t, err = t.New(parent.Name()).Parse(tpl)
 		if err != nil {
-			return "", errors.Wrapf(err, "Cannot parse template %q", tpl)
+			return "", errors.Wrapf(err, "cannot parse template %q", tpl)
 		}
 
 		var buf strings.Builder
 		if err := t.Execute(&buf, vals); err != nil {
-			return "", errors.Wrapf(err, "Error during tpl function execution for %q", tpl)
+			return "", errors.Wrapf(err, "error during tpl function execution for %q", tpl)
 		}
 
 		// See comment in renderWithReferences explaining the <no value> hack.