Skip to content

fix(ci): fix publish workflow YAML parse error#27

Merged
carlos-alm merged 1 commit intomainfrom
fix/publish-yaml-syntax
Feb 23, 2026
Merged

fix(ci): fix publish workflow YAML parse error#27
carlos-alm merged 1 commit intomainfrom
fix/publish-yaml-syntax

Conversation

@carlos-alm
Copy link
Contributor

Summary

  • Line 20 of publish.yml uses !startsWith(...) in a bare if: expression — YAML interprets the ! as a tag prefix, causing a parse error before any jobs run
  • Wraps the expression in ${{ }} so YAML sees it as a plain string

Every push to main since PR #26 was merged has been failing instantly with "You have an error in your yaml syntax on line 20".

Test plan

  • Merge and verify the next push-triggered Publish run starts jobs successfully

…rror

YAML interprets bare `!startsWith(...)` as a tag, causing the entire
workflow to fail with "error in your yaml syntax on line 20" before
any jobs run.
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 23, 2026

Greptile Summary

Fixes YAML parse error on line 20 by wrapping the if: expression in ${{ }}. The bare !startsWith(...) was interpreted by YAML as a tag prefix rather than a GitHub Actions expression, causing the workflow to fail before any jobs could run.

  • The fix correctly preserves the conditional logic: skip dev publish when either a release event occurs OR when the commit message doesn't start with 'chore: release v'
  • The ${{ }} wrapper forces YAML to treat the entire expression as a string that GitHub Actions will evaluate at runtime
  • No functional changes to the workflow logic, only syntax correction

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • Single-line syntax fix that addresses a critical YAML parsing error without any logical changes. The fix follows GitHub Actions best practices for complex conditional expressions.
  • No files require special attention

Important Files Changed

Filename Overview
.github/workflows/publish.yml Wraps if: expression in ${{ }} to fix YAML parse error caused by leading ! character

Last reviewed commit: df38b0a

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

@carlos-alm carlos-alm merged commit 7cf5da8 into main Feb 23, 2026
16 checks passed
@carlos-alm carlos-alm deleted the fix/publish-yaml-syntax branch February 23, 2026 00:59
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