mirror of
https://github.com/tiennm99/litellm.git
synced 2026-07-11 19:48:29 +00:00
Julio Quinteros Pro
4c53ccd90d
Follow-up to PR #21248 addressing greptile code review feedback. Removes hasattr checks for non-existent attributes that were identified as dead code by greptile automated code review. Changes: - Remove hasattr check for LangFuseLogger._langfuse_clients (class attribute doesn't exist) - Remove hasattr check for self.logger._langfuse_client_cache (instance attribute doesn't exist) - Update comments to be more accurate about what cleanup is being done The core fix from PR #21248 (nulling Langfuse reference and deleting logger instance) remains unchanged and effective. This just removes misleading dead code that serves no purpose. Context: These checks were added defensively but reference attributes that don't actually exist on the LangFuseLogger class, making them always no-ops. Greptile correctly identified these as dead code in PR #21248 review, but the PR was merged before the cleanup could be applied. Related: #21248 Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
In total litellm runs 1000+ tests
[02/20/2025] Update:
To make it easier to contribute and map what behavior is tested,
we've started mapping the litellm directory in tests/test_litellm
This folder can only run mock tests.