Skip to content

Commit be936da

Browse files
vaindclaude
andcommitted
fix: address review comments from seer-by-sentry bot
- Use inputs.api-token instead of github.token in Danger Docker environment - Replace $UID with $(id -u) for better portability across shell environments These changes ensure token consistency and improve cross-platform compatibility. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 7d02b06 commit be936da

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

danger/action.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,9 +32,9 @@ runs:
3232
--volume ${{ github.action_path }}:${{ github.action_path }} \
3333
--volume ${{ github.event_path }}:${{ github.event_path }} \
3434
--workdir /github/workspace \
35-
--user $UID \
35+
--user $(id -u) \
3636
-e "INPUT_ARGS" -e "GITHUB_JOB" -e "GITHUB_REF" -e "GITHUB_SHA" -e "GITHUB_REPOSITORY" -e "GITHUB_REPOSITORY_OWNER" -e "GITHUB_RUN_ID" -e "GITHUB_RUN_NUMBER" -e "GITHUB_RETENTION_DAYS" -e "GITHUB_RUN_ATTEMPT" -e "GITHUB_ACTOR" -e "GITHUB_TRIGGERING_ACTOR" -e "GITHUB_WORKFLOW" -e "GITHUB_HEAD_REF" -e "GITHUB_BASE_REF" -e "GITHUB_EVENT_NAME" -e "GITHUB_SERVER_URL" -e "GITHUB_API_URL" -e "GITHUB_GRAPHQL_URL" -e "GITHUB_REF_NAME" -e "GITHUB_REF_PROTECTED" -e "GITHUB_REF_TYPE" -e "GITHUB_WORKSPACE" -e "GITHUB_ACTION" -e "GITHUB_EVENT_PATH" -e "GITHUB_ACTION_REPOSITORY" -e "GITHUB_ACTION_REF" -e "GITHUB_PATH" -e "GITHUB_ENV" -e "GITHUB_STEP_SUMMARY" -e "RUNNER_OS" -e "RUNNER_ARCH" -e "RUNNER_NAME" -e "RUNNER_TOOL_CACHE" -e "RUNNER_TEMP" -e "RUNNER_WORKSPACE" -e "ACTIONS_RUNTIME_URL" -e "ACTIONS_RUNTIME_TOKEN" -e "ACTIONS_CACHE_URL" -e GITHUB_ACTIONS=true -e CI=true \
37-
-e GITHUB_TOKEN="${{ github.token }}" \
37+
-e GITHUB_TOKEN="${{ inputs.api-token }}" \
3838
-e DANGER_DISABLE_TRANSPILATION="true" \
3939
ghcr.io/danger/danger-js:11.3.1 \
4040
--failOnErrors --dangerfile ${{ github.action_path }}/dangerfile.js

0 commit comments

Comments
 (0)