Skip to content

Replaced Print Statements with Proper Logging#2439

Open
yzhao244 wants to merge 1 commit intoconfident-ai:mainfrom
yzhao244:patchers-fix
Open

Replaced Print Statements with Proper Logging#2439
yzhao244 wants to merge 1 commit intoconfident-ai:mainfrom
yzhao244:patchers-fix

Conversation

@yzhao244
Copy link
Contributor

Replaced Print Statements with Proper Logging

  • File: deepeval/tracing/patchers.py
  • Issue: Print statements were used instead of proper logging in production code
  • Fix: Replaced print() calls with logger.warning() using the standard logging module
  • Impact: Better control over log levels, integration with application logging, and proper error handling

Signed-off-by: Yuri Zhao <yu.zhao5@huawei.com>
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 17, 2026

Skipped: This PR was not opened by one of your configured authors: (tanayvaswani, trevor-cai, kritinv, ...)

@vercel
Copy link

vercel bot commented Jan 17, 2026

@yzhao244 is attempting to deploy a commit to the Confident AI Team on Vercel.

A member of the Team first needs to authorize it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant