Skip to content

fix: bash syntax error in deploy verify step#35

Merged
vreshch merged 1 commit intomasterfrom
fix/verify-bash-syntax
Mar 15, 2026
Merged

fix: bash syntax error in deploy verify step#35
vreshch merged 1 commit intomasterfrom
fix/verify-bash-syntax

Conversation

@vreshch
Copy link
Contributor

@vreshch vreshch commented Mar 15, 2026

Summary

  • Fix invalid bash substring expression ${LIVE_COMMIT:0:8:-no response}${LIVE_COMMIT:-no response}
  • This caused the verify step to crash instead of retrying

Test plan

  • CI passes
  • Deploy verify step completes without syntax error

@github-actions
Copy link

Pull Request Validation Successful

Step Results:

  • 📦 Install dependencies: true
  • 🔍 Run linting: true
  • 🧪 Run tests: true
  • 🏗️ Build packages: true

🎉 All quality checks passed! Ready to merge.

📋 View workflow run


🐳 Docker Images Built Successfully!

🏷️ Tag: pr-35

  • Backend: ✅ Built
  • Frontend: ✅ Built

📋 View workflow


⏰ Generated at: 2026-03-15T00:11:15.434Z

@vreshch vreshch merged commit 24d21db into master Mar 15, 2026
3 checks passed
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