Skip to content

fix(go-test): remove empty expression from comment#37

Merged
huaweic-nv merged 1 commit into
mainfrom
fix/go-test-expression-comment
Apr 9, 2026
Merged

fix(go-test): remove empty expression from comment#37
huaweic-nv merged 1 commit into
mainfrom
fix/go-test-expression-comment

Conversation

@huaweic-nv

Copy link
Copy Markdown
Contributor

Summary

GitHub Actions' template parser evaluates ${{ }} expressions even inside shell comments in run: blocks. The empty ${{ }} in the go-test action comment caused a parse failure:

Error: (Line: 114, Col: 12): An expression was expected

Fix

Replace the literal ${{ }} in the comment with plain text.

Test plan

  • auth-callout CI unit-test job passes with this fix

GitHub Actions' template parser evaluates ${{ }} expressions even
inside shell comments. The empty expression in the comment caused
"An expression was expected" parse error at line 114.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@huaweic-nv huaweic-nv merged commit ac1f695 into main Apr 9, 2026
2 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 9, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant