review.exclude_paths must be a list, crashes on string input #23

Closed
opened 2026-07-23 18:35:07 +01:00 by nimmo · 0 comments
Owner

When review.exclude_paths is set as a string (e.g. "openapi.json") instead of a list (["openapi.json"]), the action fails immediately with:

forgejo-ai-review: review.exclude_paths must be a list of path patterns

The config value should be coerced to a list (wrapping a single string), or the error message should indicate the expected format. Accepting a comma-separated string would also be a reasonable fix.

When `review.exclude_paths` is set as a string (e.g. `"openapi.json"`) instead of a list (`["openapi.json"]`), the action fails immediately with: ``` forgejo-ai-review: review.exclude_paths must be a list of path patterns ``` The config value should be coerced to a list (wrapping a single string), or the error message should indicate the expected format. Accepting a comma-separated string would also be a reasonable fix.
nimmo closed this issue 2026-07-23 19:07:32 +01:00
Sign in to join this conversation.
No labels
No milestone
No project
No assignees
1 participant
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
nimmo/forgejo-auto-pr-review#23
No description provided.