Skip to content

Require include_path to be a list#204

Merged
TideDra merged 1 commit into
mainfrom
fix-pr-192-include-path-list-only
Mar 18, 2026
Merged

Require include_path to be a list#204
TideDra merged 1 commit into
mainfrom
fix-pr-192-include-path-list-only

Conversation

@TideDra

@TideDra TideDra commented Mar 18, 2026

Copy link
Copy Markdown
Owner

Summary

  • require zotero.include_path to be a list of glob patterns or null; raise a clear error for single strings
  • match corpus items when any Zotero path matches any pattern in the list
  • update README and config comments/examples to document list syntax
  • add focused tests for validation and matching behavior

Why

This follows the review requirements on #192:

  • include_path should be and only be a list
  • docs/examples must make the list format explicit

Testing

  • uv run pytest tests/test_glob_match.py tests/test_include_path.py -q

Closes #192

@TideDra TideDra merged commit 0db4eb5 into main Mar 18, 2026
1 check passed
@TideDra TideDra deleted the fix-pr-192-include-path-list-only branch March 25, 2026 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant