🐛 fix: forward github.event_path to Docker container action #48

Merged

coding-agent-marvin8 merged 0 commits from refs/pull/48/head into main 2026-07-08 00:03:56 +00:00

Conversation 0 Commits - Files changed -

coding-agent-marvin8 commented 2026-07-07 23:28:31 +00:00 (Migrated from codeberg.org)

Copy link

When the forgejo_event_path input is not supplied, WUMING_EVENT_PATH in the action.yml env block resolved to "" and GITHUB_EVENT_PATH was not automatically inherited by the Docker container. This left pr_index empty on genuine pull_request events, causing WuMing to exit immediately with 'nothing to review'.

Fix: use ${{ inputs.forgejo_event_path || github.event_path }} so the runner-provided event path is used when the caller omits the input.

Closes #47

When the `forgejo_event_path` input is not supplied, `WUMING_EVENT_PATH` in the `action.yml` env block resolved to `""` and `GITHUB_EVENT_PATH` was not automatically inherited by the Docker container. This left `pr_index` empty on genuine `pull_request` events, causing WuMing to exit immediately with 'nothing to review'. Fix: use `${{ inputs.forgejo_event_path || github.event_path }}` so the runner-provided event path is used when the caller omits the input. Closes #47

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Reference

marvin8/wuming!48

No description provided.