cleanup

pipeline/frontend/yaml/stepbuilder/step_builder.go

@@ -50,14 +50,6 @@ type StepBuilder struct {
 	DefaultLabels        map[string]string
 }
 
-type Item struct {
-	Workflow  *model.Workflow
-	Labels    map[string]string
-	DependsOn []string
-	RunsOn    []string
-	Config    *backend_types.Config
-}
-
 func (b *StepBuilder) Build() (items []*Item, errorsAndWarnings error) {
 	b.Yamls = forge_types.SortByName(b.Yamls)
 
@@ -201,7 +193,7 @@ func (b *StepBuilder) compileWorkflow(parsed *yaml_types.Workflow, environ map[s
 		compiler.WithEnviron(environ),
 		compiler.WithEnviron(b.Envs),
 		compiler.WithEscalated(b.PrivilegedPlugins...),
-		compiler.WithTrustedClonePlugins(b.TrustedClonePlugins), // TODO: append repo trusted clone plugins
+		compiler.WithTrustedClonePlugins(b.TrustedClonePlugins),
 		compiler.WithPrefix(
 			fmt.Sprintf(
 				"wp_%s_%d",

pipeline/frontend/yaml/stepbuilder/types.go

@@ -1,8 +1,8 @@
 package stepbuilder
 
 import (
-	backend_types "go.woodpecker-ci.org/woodpecker/v2/pipeline/backend/types"
-	"go.woodpecker-ci.org/woodpecker/v2/server/model"
+	backend_types "go.woodpecker-ci.org/woodpecker/v3/pipeline/backend/types"
+	"go.woodpecker-ci.org/woodpecker/v3/server/model"
 )
 
 type Item struct {

server/pipeline/items.go

@@ -66,21 +66,6 @@ func parsePipeline(forge forge.Forge, store store.Store, currentPipeline *model.
 		})
 	}
 
-	var secrets []compiler.Secret
-	for _, sec := range secs {
-		var events []string
-		for _, event := range sec.Events {
-			events = append(events, string(event))
-		}
-
-		secrets = append(secrets, compiler.Secret{
-			Name:           sec.Name,
-			Value:          sec.Value,
-			AllowedPlugins: sec.Images,
-			Events:         events,
-		})
-	}
-
 	registryService := server.Config.Services.Manager.RegistryServiceFromRepo(repo)
 	regs, err := registryService.RegistryListPipeline(repo, currentPipeline)
 	if err != nil {
@@ -95,15 +80,6 @@ func parsePipeline(forge forge.Forge, store store.Store, currentPipeline *model.
 		})
 	}
 
-	var registries []compiler.Registry
-	for _, reg := range regs {
-		registries = append(registries, compiler.Registry{
-			Hostname: reg.Address,
-			Username: reg.Username,
-			Password: reg.Password,
-		})
-	}
-
 	if envs == nil {
 		envs = map[string]string{}
 	}

server/pipeline/metadata/metadata.go

@@ -33,15 +33,15 @@ func MetadataFromStruct(forge metadata.ServerForge, repo *model.Repo, pipeline,
 	}
 
 	fForge := metadata.Forge{}
-	if m.forge != nil {
+	if forge != nil {
 		fForge = metadata.Forge{
-			Type: m.forge.Name(),
-			URL:  m.forge.URL(),
+			Type: forge.Name(),
+			URL:  forge.URL(),
 		}
 	}
 
 	fRepo := metadata.Repo{}
-	if m.repo != nil {
+	if repo != nil {
 		fRepo = metadata.Repo{
 			ID:          repo.ID,
 			Name:        repo.Name,
@@ -53,18 +53,18 @@ func MetadataFromStruct(forge metadata.ServerForge, repo *model.Repo, pipeline,
 			Private:     repo.IsSCMPrivate,
 			Branch:      repo.Branch,
 			Trusted: metadata.TrustedConfiguration{
-				Network:  m.repo.Trusted.Network,
-				Volumes:  m.repo.Trusted.Volumes,
-				Security: m.repo.Trusted.Security,
+				Network:  repo.Trusted.Network,
+				Volumes:  repo.Trusted.Volumes,
+				Security: repo.Trusted.Security,
 			},
 		}
 
-		if idx := strings.LastIndex(m.repo.FullName, "/"); idx != -1 {
-			if fRepo.Name == "" && m.repo.FullName != "" {
-				fRepo.Name = m.repo.FullName[idx+1:]
+		if idx := strings.LastIndex(repo.FullName, "/"); idx != -1 {
+			if fRepo.Name == "" && repo.FullName != "" {
+				fRepo.Name = repo.FullName[idx+1:]
 			}
-			if fRepo.Owner == "" && m.repo.FullName != "" {
-				fRepo.Owner = m.repo.FullName[:idx]
+			if fRepo.Owner == "" && repo.FullName != "" {
+				fRepo.Owner = repo.FullName[:idx]
 			}
 		}
 	}