Skip to content

Commit 6ef40c6

Browse files
authored
Fix quoting issue in deploy-staging.yml to pass shellcheck linting
- Added double quotes around environment variables and file paths in the curl command within deploy-staging.yml. - Resolved shellcheck SC2086 error by quoting $STAGING_API_URL and @/dist/index.js to prevent globbing and word splitting. - Ensures compatibility with shellcheck and improves the reliability of the staging deployment step. This update addresses linter errors, allowing the CI workflow to proceed without deployment command issues.
1 parent 93e7bbc commit 6ef40c6

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

.github/workflows/deploy-staging.yml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@ on:
44
push:
55
pull_request:
66

7-
87
jobs:
98
build-and-deploy:
109
name: Build and Deploy to Staging
@@ -32,4 +31,4 @@ jobs:
3231
STAGING_API_URL: ${{ secrets.STAGING_API_URL }}
3332
run: |
3433
# Replace with your deployment command or script
35-
curl -X POST -H "Authorization: Bearer $STAGING_API_KEY" -d @dist/index.js $STAGING_API_URL
34+
curl -X POST -H "Authorization: Bearer $STAGING_API_KEY" -d "@/dist/index.js" "$STAGING_API_URL"

0 commit comments

Comments
 (0)