Skip to content

Commit 799386b

Browse files
chmouelpipelines-as-code[bot]
authored andcommitted
Fix trigger_comment newline
Fix yaml in prow.yaml PR or it would cause issues to the controller not being able to parse it. The pac controller would get: 🚨 11:31:31 pac-controller prun: tekton validation error: yaml: line 16: could not find expected ':' it's all deep down inside the decoder method: decoder := k8scheme.Codecs.UniversalDeserializer() so nothing we can really do on our side, and we have to tell our users to have to use the |- thing instead. Signed-off-by: Chmouel Boudjnah <[email protected]>
1 parent dcae131 commit 799386b

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

.tekton/prow.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ metadata:
99
spec:
1010
params:
1111
- name: trigger_comment
12-
value: |
12+
value: |-
1313
{{ trigger_comment }}
1414
- name: repo_owner
1515
value: "{{ repo_owner }}"

docs/content/docs/guide/authoringprs.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ You will encounter a YAML validation error because objects or multiline strings
8787
spec:
8888
params:
8989
- name: body
90-
value: |
90+
value: |-
9191
{{ body }}
9292
# Alternatively, use '>' to specify that the value will be in block format
9393
- name: pull_request

0 commit comments

Comments
 (0)