Deprecate "platform" filter in favour of "labels"

docs/docs/20-usage/20-pipeline-syntax.md

@@ -603,26 +603,6 @@ Woodpecker has integrated support for matrix builds. Woodpecker executes a separ
 
 For more details check the [matrix build docs](./30-matrix-workflows.md).
 
-## `platform`
-
-To configure your pipeline to only be executed on an agent with a specific platform, you can use the `platform` key.
-Have a look at the official [go docs](https://go.dev/doc/install/source) for the available platforms. The syntax of the platform is `GOOS/GOARCH` like `linux/arm64` or `linux/amd64`.
-
-Example:
-
-Assuming we have two agents, one `arm` and one `amd64`. Previously this pipeline would have executed on **either agent**, as Woodpecker is not fussy about where it runs the pipelines. By setting the following option it will only be executed on an agent with the platform `linux/arm64`.
-
-```diff
-+platform: linux/arm64
-
-steps:
-  build:
-    image: golang
-    commands:
-      - go build
-      - go test
-```
-
 ## `labels`
 
 You can set labels for your pipeline to select an agent to execute the pipeline on. An agent will pick up and run a pipeline when **every** label assigned to a pipeline matches the agents labels.
@@ -648,6 +628,23 @@ steps:
       - go test
 ```
 
+### filter by platform
+
+To configure your pipeline to only be executed on an agent with a specific platform, you can use the `platform` key.
+Have a look at the official [go docs](https://go.dev/doc/install/source) for the available platforms. The syntax of the platform is `GOOS/GOARCH` like `linux/arm64` or `linux/amd64`.
+
+Example:
+
+Assuming we have two agents, one `arm` and one `amd64`. Previously this pipeline would have executed on **either agent**, as Woodpecker is not fussy about where it runs the pipelines. By setting the following option it will only be executed on an agent with the platform `linux/arm64`.
+
+```diff
++labels:
++  platform: linux/arm64
+
+steps:
+  [...]
+```
+
 ## `variables`
 
 Woodpecker supports [YAML anchors & aliases](https://yaml.org/spec/1.2.2/#3222-anchors-and-aliases) in the pipeline configuration. These can be used as variables to not repeat yourself.

docs/docs/91-migrations.md

@@ -5,6 +5,7 @@ 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
+- Deprecate `platform:` filter in favor of `labels:`, [read more](./20-usage/20-pipeline-syntax.md#filter-by-platform)
 
 ## 1.0.0
 

pipeline/frontend/yaml/parse.go

@@ -35,6 +35,13 @@ func ParseBytes(b []byte) (*types.Workflow, error) {
 	}
 	out.PipelineDontUseIt.ContainerList = nil
 
+	// support deprecated platform filter
+	if out.PlatformDontUseIt == "" {
+		if _, set := out.Labels["platform"]; !set {
+			out.Labels["platform"] = out.PlatformDontUseIt
+		}
+	}
+
 	return out, nil
 }
 

pipeline/frontend/yaml/types/workflow.go

@@ -9,7 +9,6 @@ type (
 	// Workflow defines a workflow configuration.
 	Workflow struct {
 		When      constraint.When `yaml:"when,omitempty"`
-		Platform  string          `yaml:"platform,omitempty"`
 		Workspace Workspace       `yaml:"workspace,omitempty"`
 		Clone     ContainerList   `yaml:"clone,omitempty"`
 		Steps     ContainerList   `yaml:"steps,omitempty"`
@@ -18,10 +17,14 @@ type (
 		DependsOn []string        `yaml:"depends_on,omitempty"`
 		RunsOn    []string        `yaml:"runs_on,omitempty"`
 		SkipClone bool            `yaml:"skip_clone"`
+
 		// Undocumented
 		Cache    base.StringOrSlice `yaml:"cache,omitempty"`
 		Networks WorkflowNetworks   `yaml:"networks,omitempty"`
 		Volumes  WorkflowVolumes    `yaml:"volumes,omitempty"`
+
+		// Deprecated
+		PlatformDontUseIt string `yaml:"platform,omitempty"` // TODO: remove after v1.2.x version
 		// Deprecated
 		BranchesDontUseIt *constraint.List `yaml:"branches,omitempty"` // TODO: remove after v1.1.x version
 		// Deprecated

pipeline/stepBuilder.go

@@ -54,7 +54,6 @@ type StepBuilder struct {
 
 type Item struct {
 	Workflow  *model.Workflow
-	Platform  string
 	Labels    map[string]string
 	DependsOn []string
 	RunsOn    []string
@@ -171,7 +170,6 @@ func (b *StepBuilder) genItemForWorkflow(workflow *model.Workflow, axis matrix.A
 		Labels:    parsed.Labels,
 		DependsOn: parsed.DependsOn,
 		RunsOn:    parsed.RunsOn,
-		Platform:  parsed.Platform,
 	}
 	if item.Labels == nil {
 		item.Labels = map[string]string{}

server/pipeline/queue.go

@@ -37,7 +37,6 @@ func queuePipeline(repo *model.Repo, pipelineItems []*pipeline.Item) error {
 		for k, v := range item.Labels {
 			task.Labels[k] = v
 		}
-		task.Labels["platform"] = item.Platform
 		task.Labels["repo"] = repo.FullName
 		task.Dependencies = taskIds(item.DependsOn, pipelineItems)
 		task.RunOn = item.RunsOn