Skip to content

Commit a5d3771

Browse files
cpcloudclaude
andcommitted
fix(ci): fix warning annotation and guard semantic-release on Go test success
Send ::warning:: to stdout (not stderr) so GitHub Actions renders it as an annotation. Gate semantic-release on test and nix-build success when Go changes are present, preventing releases without passing tests. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 0aeb3fc commit a5d3771

2 files changed

Lines changed: 2 additions & 1 deletion

File tree

.github/detect-go-changes.bash

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ fi
3232
if [ "$event_name" != "pull_request" ]; then
3333
file_count=$(echo "$changed" | wc -l)
3434
if [ "$file_count" -ge 300 ]; then
35-
echo "::warning::Compare API file cap hit ($file_count files), assuming Go changes" >&2
35+
echo "::warning::Compare API file cap hit ($file_count files), assuming Go changes"
3636
echo true
3737
exit 0
3838
fi

.github/workflows/ci.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -231,6 +231,7 @@ jobs:
231231
always()
232232
&& !contains(needs.*.result, 'failure')
233233
&& !contains(needs.*.result, 'cancelled')
234+
&& (needs.changes.outputs.go != 'true' || (needs.test.result == 'success' && needs.nix-build.result == 'success'))
234235
runs-on: ubuntu-latest
235236
concurrency:
236237
group: semantic-release-${{ github.ref }}

0 commit comments

Comments
 (0)