AbanteAI / mentat

Mentat - The AI Coding Assistant
https://mentat.ai
Apache License 2.0
2.54k stars 234 forks source link

Update requirements #567

Closed jakethekoenig closed 5 months ago

jakethekoenig commented 5 months ago

Numpy and attrs no longer pinned to exact versions. Chroma removed (used by ragdaemon now) pytest and fire dependencies moved to dev-requirements

Pull Request Checklist

mentatbot[bot] commented 5 months ago

MENTAT CODE REVIEW IN ACTIVE DEVELOPMENT. Only in use on mentat and internal repos. Please Reply with feedback.

This PR makes several dependency management changes, including moving some dependencies to dev-requirements.txt and updating version constraints for numpy and attrs. While these changes seem to be in the right direction for maintaining a clean separation of development and production dependencies, it would be beneficial to include more details in the PR description about the testing strategy for the updated version constraints and the rationale behind moving specific dependencies to development requirements. Additionally, clarifying the handling of dependencies like chromadb and its relation to ragdaemon would help in understanding the full impact of these changes.