Skip to content

Fix GitHub Actions output parsing in sync workflow#78

Merged
agocke merged 2 commits into
bazelfrom
refactor/testable-copilot-autofix
Mar 14, 2026
Merged

Fix GitHub Actions output parsing in sync workflow#78
agocke merged 2 commits into
bazelfrom
refactor/testable-copilot-autofix

Conversation

@agocke
Copy link
Copy Markdown
Owner

@agocke agocke commented Mar 14, 2026

The sync script output may contain \r\n line endings, causing values like classification=build-changes\r to be written to $GITHUB_OUTPUT. GitHub Actions rejects this as Invalid format 'build-changes'.

Fix: add tr -d '\r' when parsing Branch, Classification, and PR fields from script output.

agocke and others added 2 commits March 14, 2026 08:02
Under set -euo pipefail, git rev-list | head -1 causes SIGPIPE when
head closes the pipe before git finishes writing.

Fix: capture full output into a variable, use here-strings to extract
the first line and count.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
The sync script output may contain \r\n line endings, causing
classification=build-changes\r to be written to GITHUB_OUTPUT.
GitHub Actions rejects this as invalid format.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@agocke agocke merged commit 926c675 into bazel Mar 14, 2026
4 checks passed
@agocke agocke deleted the refactor/testable-copilot-autofix branch March 14, 2026 17:50
agocke added a commit that referenced this pull request May 19, 2026
> [!NOTE]
> This PR description was generated with AI/Copilot assistance.

## Summary
- add BuildXL workspace files for the TieredCompilation CoreCLR test
slice
- switch the BuildXL entrypoint to discover BuildXL and dotnet from the
local environment
- add a GitHub Actions workflow that runs the BuildXL build on pushes
and PRs targeting `main` and `bxl`

## Validation
- `DOTNET_ROOT=/home/andy/.local/share/dnvm/dn
DOTNET_SDK_VERSION=11.0.100-preview.3.26207.106 ./bxl.sh
/fileVerbosity:Informational /consoleVerbosity:Informational`

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

1 participant