Skip to content

Commit 9b5fb63

Browse files
committed
fix: correct heredoc syntax in integrate-develop workflow
- Split PR body creation to handle multi-line commits properly - Avoid variable expansion issues in heredoc - Remove test file
1 parent fad2ac6 commit 9b5fb63

File tree

2 files changed

+9
-4
lines changed

2 files changed

+9
-4
lines changed

.github/workflows/integrate-develop.yml

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -77,8 +77,8 @@ jobs:
7777
run: |
7878
COMMIT_COUNT="${{ steps.commit-info.outputs.commit_count }}"
7979
TIMESTAMP="${{ steps.commit-info.outputs.timestamp }}"
80-
COMMITS="${{ steps.commit-info.outputs.commits }}"
8180
81+
# Write PR body
8282
cat > pr-body.md << EOF
8383
## 🚀 Integration from develop to staging
8484

@@ -88,7 +88,13 @@ jobs:
8888
- **Auto-generated**: This PR was automatically created by the integration workflow
8989

9090
### 📝 Recent Commits
91-
${COMMITS}
91+
EOF
92+
93+
# Append commits safely
94+
echo "${{ steps.commit-info.outputs.commits }}" >> pr-body.md
95+
96+
# Append rest of PR body
97+
cat >> pr-body.md << 'EOF'
9298

9399
### 🔍 What happens next?
94100
1. Review the changes in this PR
@@ -134,7 +140,7 @@ EOF
134140
**New commits**: ${COMMIT_COUNT}
135141

136142
### 📝 Recent Commits
137-
${COMMITS}"
143+
$COMMITS"
138144

139145
# Update or append the updated section
140146
if echo "$CURRENT_BODY" | grep -q "### 🔄 Last Updated:"; then

test-workflow.md

Lines changed: 0 additions & 1 deletion
This file was deleted.

0 commit comments

Comments
 (0)