make pydantic optional#6786
Conversation
Merging this PR will not alter performance
Comparing Footnotes
|
Greptile SummaryThis PR makes pydantic an optional dependency for both
Confidence Score: 5/5Safe to merge; pydantic is cleanly demoted to an optional dep with a well-exercised test covering the no-pydantic path. All changed files are straightforward: two pyproject.toml edits that move pydantic to an optional extra, a lock file update, and a new subprocess test verifying the no-pydantic path. The db extra still pulls pydantic for database users, so existing workflows are unaffected. No files require special attention. Important Files Changed
Reviews (4): Last reviewed commit: "greptile" | Re-trigger Greptile |
masenf
left a comment
There was a problem hiding this comment.
interesting; i thought we had pydantic as optional for some time now, but i guess not...
I remember reading some git commits or changelogs which mentioned this as well - however somehow it currently isn't optional. I guess this is ready for review now. |
No description provided.