1
0
Fork
You've already forked wuming
0

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

Manually merged
marvin8 merged 1 commit from fix/issue-47-event-path into main 2026年07月08日 02:03:56 +02:00

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
🐛 fix: forward github.event_path to Docker container action
All checks were successful
ci/woodpecker/pr/ai-review Pipeline was successful
/ ai-review (pull_request) Successful in 22s
ci/woodpecker/pr/ci Pipeline was successful
b491222a98
 
When forgejo_event_path input is not supplied, the WUMING_EVENT_PATH
env var resolved to empty string and GITHUB_EVENT_PATH was not
inherited by the Docker container, leaving pr_index empty and causing
an early exit with 'nothing to review' on real PR events.
Closes #47
marvin8 manually merged commit 916aaaf237 into main 2026年07月08日 02:03:56 +02:00
marvin8 deleted branch fix/issue-47-event-path 2026年07月08日 02:05:06 +02:00
Sign in to join this conversation.
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
marvin8/wuming!48
Reference in a new issue
marvin8/wuming
No description provided.
Delete branch "fix/issue-47-event-path"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?