Skip to content

ci: fix update-generated-files workflow - #5006

Merged
straker merged 3 commits into
developfrom
straker-patch-1
Feb 3, 2026
Merged

ci: fix update-generated-files workflow#5006
straker merged 3 commits into
developfrom
straker-patch-1

Conversation

@straker

@straker straker commented Feb 2, 2026

Copy link
Copy Markdown
Contributor

It would seem that the GITHUB_OUTPUT syntax does not allow line breaks. Since the output to git status --porcelain produces a line with 2 changes

 M doc/rule-descriptions.md
 M locales/_template.json

removing the line breaks from the variable should fix the problem.

It would seem that the `GITHUB_OUTPUT` syntax does not allow line breaks. Since the output to `git status --porcelain` produces a line with 2 changes

```
 M doc/rule-descriptions.md
 M locales/_template.json
```

removing the line breaks from the variable should fix the problem.
@straker
straker requested a review from a team as a code owner February 2, 2026 21:19
@straker
straker merged commit 835e930 into develop Feb 3, 2026
10 of 19 checks passed
@straker
straker deleted the straker-patch-1 branch February 3, 2026 16:38
WilcoFiers pushed a commit that referenced this pull request Mar 30, 2026
It would seem that the `GITHUB_OUTPUT` syntax [does not allow line
breaks](https://github.com/orgs/community/discussions/106666). Since the
output to `git status --porcelain` produces a line with 2 changes

```
 M doc/rule-descriptions.md
 M locales/_template.json
```

removing the line breaks from the variable should fix the problem.
straker added a commit that referenced this pull request Mar 30, 2026
It would seem that the `GITHUB_OUTPUT` syntax [does not allow line
breaks](https://github.com/orgs/community/discussions/106666). Since the
output to `git status --porcelain` produces a line with 2 changes

```
 M doc/rule-descriptions.md
 M locales/_template.json
```

removing the line breaks from the variable should fix the problem.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants