Skip to content

Commit 60d174a

Browse files
committed
use $GITHUB_WORKSPACE (container-aware) not ${{ github.workspace }}
1 parent 3035851 commit 60d174a

File tree

3 files changed

+6
-4
lines changed

3 files changed

+6
-4
lines changed

actions/cache/action.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,6 @@ runs:
2323
path: |
2424
~/.deno
2525
${{ steps.os-cache.outputs.cache }}
26-
# This isn't perfect (can't hash stuff outside github.workspace, and if the there scripts change, the hash won't)
26+
# This isn't perfect (can't hash stuff outside $GITHUB_WORKSPACE, and if the there scripts change, the hash won't)
2727
# but it's good enough for now. It's slightly conservative, since it monitors all .ts files, but that's fine.
2828
key: ${{ runner.os }}-deno-${{ inputs.cache-name }}-${{ hashFiles('**/deno.jsonc', '**/*.ts') }}

actions/get-platform/action.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,4 +46,4 @@ runs:
4646
env:
4747
PLATFORM: ${{ inputs.platform }}
4848
PROJECTS: ${{ inputs.projects }}
49-
PKGX_PANTRY_PATH: ${{ github.workspace }}
49+
PKGX_PANTRY_PATH: ${{ env.GITHUB_WORKSPACE }}

actions/setup-brewkit/action.yml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,9 +46,11 @@ runs:
4646
with:
4747
cache-name: setup
4848

49+
# $GITHUB_WORKSPACE is correct when using a container
50+
# ${{ github.workspace }} is the runner
4951
- run: |
50-
if test -d "${{ github.workspace }}"/projects; then
51-
echo "PKGX_PANTRY_PATH=${{ github.workspace }}" >> $GITHUB_ENV
52+
if test -d "$GITHUB_WORKSPACE"/projects; then
53+
echo "PKGX_PANTRY_PATH=$GITHUB_WORKSPACE" >> $GITHUB_ENV
5254
fi
5355
shell: bash
5456

0 commit comments

Comments
 (0)