Skip to content

Commit 151fafe

Browse files
authored
Merge pull request #430 from netwrix/dev
fix empty expression in shell comment and footer formatting
2 parents 917416d + 425272a commit 151fafe

1 file changed

Lines changed: 9 additions & 9 deletions

File tree

.github/workflows/claude-doc-pr.yml

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -97,14 +97,14 @@ jobs:
9797
run: |
9898
REVIEW_FILE="/tmp/doc-pr-review.md"
9999
FOOTER=$'\n\n---\n\n'
100-
FOOTER+='**What to do next:**\n\n'
101-
FOOTER+='Comment `@claude` on this PR followed by your instructions to get help:\n\n'
102-
FOOTER+='- `@claude fix all issues` — fix all Vale, Dale, and editorial issues\n'
103-
FOOTER+='- `@claude fix only the Vale issues` — fix just the linting problems\n'
104-
FOOTER+='- `@claude help improve the flow of this document` — get writing assistance\n'
105-
FOOTER+='- `@claude explain the voice issues` — understand why something was flagged\n\n'
106-
FOOTER+='You can ask Claude anything about the review or about Netwrix writing standards.\n\n'
107-
FOOTER+='> Automated fixes are only available for branches in this repository, not forks.'
100+
FOOTER+=$'**What to do next:**\n\n'
101+
FOOTER+=$'Comment `@claude` on this PR followed by your instructions to get help:\n\n'
102+
FOOTER+=$'- `@claude fix all issues` — fix all Vale, Dale, and editorial issues\n'
103+
FOOTER+=$'- `@claude fix only the Vale issues` — fix just the linting problems\n'
104+
FOOTER+=$'- `@claude help improve the flow of this document` — get writing assistance\n'
105+
FOOTER+=$'- `@claude explain the voice issues` — understand why something was flagged\n\n'
106+
FOOTER+=$'You can ask Claude anything about the review or about Netwrix writing standards.\n\n'
107+
FOOTER+=$'> Automated fixes are only available for branches in this repository, not forks.'
108108
109109
if [ -f "$REVIEW_FILE" ]; then
110110
BODY=$(cat "$REVIEW_FILE")
@@ -147,7 +147,7 @@ jobs:
147147
echo "base_branch=$BASE_BRANCH" >> "$GITHUB_OUTPUT"
148148
echo "is_fork=$(echo "$PR_DATA" | jq -r '.isCrossRepository')" >> "$GITHUB_OUTPUT"
149149
# Check target branch here using the shell variable to avoid
150-
# re-interpolating the output via ${{ }} (code injection risk).
150+
# re-interpolating the output via expressions (code injection risk).
151151
if [ "$BASE_BRANCH" = "dev" ]; then
152152
echo "targets_dev=true" >> "$GITHUB_OUTPUT"
153153
else

0 commit comments

Comments
 (0)