Skip to content

Commit 8694d67

Browse files
themr0cclaude
andcommitted
fix: improve deploy step naming and inline trap cleanup
- Rename "Deploy to gh-pages" to "Deploy to the gh-pages branch" - Inline the trap cleanup to eliminate shellcheck SC2329 suppression Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 921fc74 commit 8694d67

2 files changed

Lines changed: 2 additions & 4 deletions

File tree

.github/workflows/build-asciidoc.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ jobs:
5050
echo "Building branch ${{ env.GIT_BRANCH }}"
5151
build/scripts/build-ccutil.sh -b ${{ env.GIT_BRANCH }}
5252
53-
- name: Deploy to gh-pages
53+
- name: Deploy to the gh-pages branch
5454
env:
5555
GITHUB_TOKEN: ${{ secrets.RHDH_BOT_TOKEN }}
5656
GITHUB_REPOSITORY: ${{ github.repository }}

build/scripts/deploy-gh-pages.sh

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,9 +36,7 @@ MAX_RETRIES=3
3636
RETRY_DELAYS=(0 27 133)
3737

3838
DEPLOY_DIR="$(mktemp -d)"
39-
# shellcheck disable=SC2329 # invoked via trap
40-
cleanup() { rm -rf "$DEPLOY_DIR"; }
41-
trap cleanup EXIT
39+
trap 'rm -rf "$DEPLOY_DIR"' EXIT
4240

4341
cd "$DEPLOY_DIR"
4442
git init -q

0 commit comments

Comments
 (0)