Fail on deprecated pipeline keys

docs/docs/91-migrations.md

@@ -5,6 +5,8 @@ Some versions need some changes to the server configuration or the pipeline conf
 ## next (1.1.0)
 
 - Drop deprecated `CI_BUILD_*`, `CI_PREV_BUILD_*`, `CI_JOB_*`, `*_LINK`, `CI_SYSTEM_ARCH`, `CI_REPO_REMOTE` built-in environment variables
+- Drop deprecated `pipeline:` keyword for steps in yaml config
+- Drop deprecated `branches:` keyword for global branch filter
 
 ## 1.0.0
 

pipeline/frontend/yaml/parse.go

@@ -5,7 +5,6 @@ import (
 
 	"codeberg.org/6543/xyaml"
 
-	"github.com/woodpecker-ci/woodpecker/pipeline/frontend/yaml/constraint"
 	"github.com/woodpecker-ci/woodpecker/pipeline/frontend/yaml/types"
 )
 
@@ -17,23 +16,15 @@ func ParseBytes(b []byte) (*types.Workflow, error) {
 		return nil, err
 	}
 
-	// support deprecated branch filter
+	// fail hard on deprecated branch filter
 	if out.BranchesDontUseIt != nil {
-		if out.When.Constraints == nil {
-			out.When.Constraints = []constraint.Constraint{{Branch: *out.BranchesDontUseIt}}
-		} else if len(out.When.Constraints) == 1 && out.When.Constraints[0].Branch.IsEmpty() {
-			out.When.Constraints[0].Branch = *out.BranchesDontUseIt
-		} else {
-			return nil, fmt.Errorf("could not apply deprecated branches filter into global when filter")
-		}
-		out.BranchesDontUseIt = nil
+		return nil, fmt.Errorf("\"branches:\" filter got removed, use \"branch\" in global when filter")
 	}
 
-	// support deprecated pipeline keyword
-	if len(out.PipelineDontUseIt.ContainerList) != 0 && len(out.Steps.ContainerList) == 0 {
-		out.Steps.ContainerList = out.PipelineDontUseIt.ContainerList
+	// fail hard on deprecated pipeline keyword
+	if len(out.PipelineDontUseIt.ContainerList) != 0 {
+		return nil, fmt.Errorf("\"pipeline:\" got removed, user \"steps:\"")
 	}
-	out.PipelineDontUseIt.ContainerList = nil
 
 	return out, nil
 }