chore: update according to steinliber's review

Signed-off-by: Bird <aflybird0@gmail.com>

Author: aFlyBird0

internal/pkg/configmanager/rawconfig.go

@@ -7,8 +7,6 @@ import (
 	"unicode"
 
 	"gopkg.in/yaml.v3"
-
-	"github.com/devstream-io/devstream/pkg/util/log"
 )
 
 // rawConfig respent every valid config block for devstream
@@ -138,7 +136,6 @@ func (c *rawConfig) getTemplatePipelineMap() (map[string]string, error) {
 		if _, ok := pipelineTemplateMap[t.Name]; ok {
 			return nil, fmt.Errorf("pipelineTemplate <%s> is duplicated", t.Name)
 		}
-		log.Infof("pipelineTemplate %s is loaded", t.Name)
 		pipelineTemplateMap[t.Name] = string(rawPipeline)
 	}
 	return pipelineTemplateMap, nil

internal/pkg/configmanager/tool.go

@@ -66,7 +66,7 @@ func (tools Tools) validate() (errs []error) {
 	for _, tool := range tools {
 		errs = append(errs, tool.validate()...)
 	}
-	errs = append(errs, tools.DuplicatedCheck()...)
+	errs = append(errs, tools.duplicatedCheck()...)
 	return
 }
 
@@ -117,11 +117,7 @@ func (t *Tool) GetPluginMD5FileNameWithOSAndArch(os, arch string) string {
 	return t.GetPluginNameWithOSAndArch(os, arch) + ".md5"
 }
 
-func (t *Tool) EqualNameAndInstanceID(other *Tool) bool {
-	return t.Name == other.Name && t.InstanceID == other.InstanceID
-}
-
-func (tools Tools) DuplicatedCheck() (errs []error) {
+func (tools Tools) duplicatedCheck() (errs []error) {
 	list := make(map[string]struct{})
 	for _, t := range tools {
 		key := t.KeyWithNameAndInstanceID()

internal/pkg/configmanager/tool_test.go

@@ -166,7 +166,7 @@ var _ = Describe("Tool struct", func() {
 	})
 })
 
-var _ = Describe("DuplicatedCheck", func() {
+var _ = Describe("duplicatedCheck", func() {
 	var (
 		errs                   []error
 		tools                  Tools
@@ -184,7 +184,7 @@ var _ = Describe("DuplicatedCheck", func() {
 	)
 
 	JustBeforeEach(func() {
-		errs = tools.DuplicatedCheck()
+		errs = tools.duplicatedCheck()
 	})
 
 	When("tools has duplicated name and instanceID", func() {