Skip to content

Commit 6a4bf92

Browse files
Fix actionlint: move ALLOWED_RULES_FILE from job-level to step-level env
The runner context is not available in job-level env blocks. Use $RUNNER_TEMP directly in the extract step's run command, and set ALLOWED_RULES_FILE via step-level env on the post step where the runner context is valid.
1 parent 5235d6e commit 6a4bf92

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

.github/workflows/claude-review.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@ jobs:
2929
runs-on: ubuntu-latest
3030
env:
3131
PR_NUMBER: ${{ github.event.pull_request.number }}
32-
ALLOWED_RULES_FILE: ${{ runner.temp }}/allowed-rules.txt
3332
steps:
3433
- name: Checkout repository
3534
uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6
@@ -58,7 +57,7 @@ jobs:
5857
run: |
5958
"$GITHUB_WORKSPACE/.github/scripts/extractAllowedRules.sh" \
6059
"$GITHUB_WORKSPACE/.claude/skills/coding-standards/rules" \
61-
"$ALLOWED_RULES_FILE"
60+
"$RUNNER_TEMP/allowed-rules.txt"
6261
6362
- name: Load code review JSON schema
6463
id: schema
@@ -83,6 +82,7 @@ jobs:
8382
env:
8483
GH_TOKEN: ${{ github.token }}
8584
STRUCTURED_OUTPUT: ${{ steps.code-review.outputs.structured_output }}
85+
ALLOWED_RULES_FILE: ${{ runner.temp }}/allowed-rules.txt
8686
run: |
8787
if [ -z "$STRUCTURED_OUTPUT" ]; then
8888
echo "::error::Claude Code Action returned empty structured output"

0 commit comments

Comments
 (0)