Skip to content

Conversation

@hassiebp
Copy link
Contributor

@hassiebp hassiebp commented Mar 20, 2025

Important

Remove unused import openai.resources from openai.py.

  • Code Cleanup:
    • Removed unused import openai.resources from openai.py.

This description was created by Ellipsis for 737903c. It will automatically update as commits are pushed.

Greptile Summary

Disclaimer: Experimental PR review

Removed unused openai.resources import from langfuse/openai.py, cleaning up the codebase while maintaining full OpenAI integration functionality.

  • Removed import openai.resources as it's not directly used - specific imports like openai.resources.chat.completions are used instead in OPENAI_METHODS_V1
  • No functional changes as the OpenAI integration continues to work through explicit resource imports

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile

@hassiebp hassiebp linked an issue Mar 20, 2025 that may be closed by this pull request
@hassiebp hassiebp merged commit e242d62 into main Mar 21, 2025
10 checks passed
@hassiebp hassiebp deleted the hassieb/lfe-4472-openai-remove-unused-import branch March 21, 2025 16:51
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.

bug: No module named 'openai.resources'

2 participants