fix(ollama): remove unused testcontainers dependency#7013
fix(ollama): remove unused testcontainers dependency#7013NIK-TIGER-BILL wants to merge 1 commit into
Conversation
The @chroma-core/ollama package declared a dependency on 'testcontainers' in its package.json, but the package never actually imports or uses it. Removing the unused dependency avoids unnecessary transitive installs and resolves the circular-dependency problem reported in chroma-core#6972. Signed-off-by: NIK-TIGER-BILL <nik.tiger.bill@github.com>
Reviewer ChecklistPlease leverage this checklist to ensure your code review is thorough before approving Testing, Bugs, Errors, Logs, Documentation
System Compatibility
Quality
|
There was a problem hiding this comment.
Review found no issues; removing the unused dependency is appropriate and low risk.
Status: No Issues Found | Risk: Low
Review Details
📁 1 files reviewed | 💬 0 comments
Instruction Files
├── .claude/
│ └── rules/
│ └── foundation-rust.md
├── AGENTS.md
└── CLAUDE.md
|
Remove unused This PR updates The change aligns with the stated intent to eliminate an unnecessary transitive dependency and address the circular-dependency issue referenced in This summary was automatically generated by @propel-code-bot |
Closing this PR as it has been open for 11+ days with only bot approval and no human maintainer review or comments. Will revisit if still relevant.