Skip to content

Commit 800ec51

Browse files
fix: extract build artifact in tmp directory, move installation step earlier (#22306)
Co-authored-by: Grace Park <gracepark@github.com>
1 parent 72e5015 commit 800ec51

1 file changed

Lines changed: 12 additions & 11 deletions

File tree

.github/workflows/staging-deploy-pr.yml

Lines changed: 12 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -303,6 +303,10 @@ jobs:
303303
name: Install temporary dependencies
304304
run: npm install --no-save dotenv rimraf
305305

306+
# Install any additional dependencies *before* downloading the build artifact
307+
- name: Install Heroku client development-only dependency
308+
run: npm install --no-save heroku-client
309+
306310
- if: ${{ github.repository == 'github/docs-internal' }}
307311
name: Clone early access
308312
run: node script/early-access/clone-for-build.js
@@ -317,31 +321,28 @@ jobs:
317321
workflow: ${{ github.event.workflow_run.workflow_id }}
318322
run_id: ${{ env.BUILD_ACTIONS_RUN_ID }}
319323
name: pr_build
320-
path: ./
324+
path: ${{ env.RUNNER_TEMP }}
321325

322326
- name: Extract user-changes to temp directory
323327
run: |
324-
tar -x --file=app.tar -C "$RUNNER_TEMP/"
325-
rm app.tar
328+
mkdir $RUNNER_TEMP/app
329+
tar -x --file=$RUNNER_TEMP/app.tar -C "$RUNNER_TEMP/app/"
326330
327331
# Move the LFS content into the temp directory in chunks (destructively)
328332
- name: Move the LFS objects
329333
run: |
330-
git lfs ls-files --name-only | xargs -n 1 -I {} sh -c 'mkdir -p "$RUNNER_TEMP/$(dirname {})"; mv {} "$RUNNER_TEMP/$(dirname {})/"'
334+
git lfs ls-files --name-only | xargs -n 1 -I {} sh -c 'mkdir -p "$RUNNER_TEMP/app/$(dirname {})"; mv {} "$RUNNER_TEMP/app/$(dirname {})/"'
331335
332336
# Move the early access content into the temp directory (destructively)
333337
- if: ${{ github.repository == 'github/docs-internal' }}
334338
name: Move the early access content
335339
run: |
336-
mv assets/images/early-access "$RUNNER_TEMP/assets/images/"
337-
mv content/early-access "$RUNNER_TEMP/content/"
338-
mv data/early-access "$RUNNER_TEMP/data/"
340+
mv assets/images/early-access "$RUNNER_TEMP/app/assets/images/"
341+
mv content/early-access "$RUNNER_TEMP/app/content/"
342+
mv data/early-access "$RUNNER_TEMP/app/data/"
339343
340344
- name: Create a gzipped archive
341-
run: tar -cz --file app.tar.gz "$RUNNER_TEMP/"
342-
343-
- name: Install Heroku client development-only dependency
344-
run: npm install --no-save heroku-client
345+
run: tar -cz --file app.tar.gz "$RUNNER_TEMP/app/"
345346

346347
- name: Create a Heroku build source
347348
id: build-source

0 commit comments

Comments
 (0)