Skip to content

pin litellm to sub 1.97 for 3.10 to resolve dependency issue with crewAI tests in 3.10 unit tests - #864

Merged
haneric00 merged 1 commit into
mainfrom
310-unittest-fix
Aug 25, 2026
Merged

pin litellm to sub 1.97 for 3.10 to resolve dependency issue with crewAI tests in 3.10 unit tests#864
haneric00 merged 1 commit into
mainfrom
310-unittest-fix

Conversation

@haneric00

Copy link
Copy Markdown
Contributor

CrewAI had a chage in dependencies which means it no longer interfaces with the most recent version of litellm. The next working version of litellm that interfaces with the most current version of CrewAI is 1.96.x so we temporarily pin the version of litellm to be that.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@haneric00
haneric00 requested a review from a team as a code owner August 25, 2026 20:13
@haneric00 haneric00 added the skip changelog doesn't need a CHANGELOG entry label Aug 25, 2026
@github-actions

github-actions Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Claude finished @haneric00's task in 1m 22s —— View job


Claude PR Review

  • Read prior review comments to dedupe (none found)
  • Review PR diff
  • Post findings

Findings

No blocking issues. This is a well-scoped, well-documented change:

  • The pin uses a correct PEP 508 environment marker (python_version == "3.10"), so 3.11+ CI keeps testing against the latest litellm as intended.
  • <1.97.0 correctly excludes both breakages described in the comment (the 1.97.0 unresolved-forward-ref on Message, and the 1.98.0 NotRequired-from-typing import).
  • The inline comment does exactly what a version pin comment should do: names the two distinct upstream failure modes, states which Python versions are affected, records the newest usable version (1.96.2), points at the upstream project (BerriAI/litellm), and gives a clear "lift this cap when…" condition. Future maintainers will not have to re-derive any of this.
  • requirements.oldest.txt doesn't need the same pin — it's pinned to crewai==1.10.0, which resolves its own compatible litellm range.

Nothing to change.
· Branch: 310-unittest-fix

@haneric00
haneric00 merged commit d1b0a33 into main Aug 25, 2026
493 of 498 checks passed
@haneric00
haneric00 deleted the 310-unittest-fix branch August 25, 2026 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip changelog doesn't need a CHANGELOG entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants