Skip to content

Fix #325: Make ConfigRegistryAccessor::getDocuments() return an empty DocumentRegistry + Add DocumentRegistry::hasItems()#326

Merged
butschster merged 1 commit intocontext-hub:mainfrom
vjik:fix-empty-documents
Mar 6, 2026
Merged

Fix #325: Make ConfigRegistryAccessor::getDocuments() return an empty DocumentRegistry + Add DocumentRegistry::hasItems()#326
butschster merged 1 commit intocontext-hub:mainfrom
vjik:fix-empty-documents

Conversation

@vjik
Copy link
Copy Markdown
Contributor

@vjik vjik commented Mar 6, 2026

No description provided.

…ntRegistry` instead of `null` to prevent `TypeError` in `foreach` loop + Add `DocumentRegistry::hasItems()`
@butschster butschster merged commit faf9a48 into context-hub:main Mar 6, 2026
6 checks passed
@butschster butschster self-assigned this Mar 6, 2026
@butschster butschster added bug Something isn't working config:loader ConfigLoader component for parsing and validating config files labels Mar 6, 2026
@butschster butschster added this to the 1.34 milestone Mar 6, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@vjik vjik deleted the fix-empty-documents branch March 13, 2026 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working config:loader ConfigLoader component for parsing and validating config files

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants