Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix typo #268

Merged
merged 1 commit into from
Apr 4, 2024
Merged

Fix typo #268

merged 1 commit into from
Apr 4, 2024

Conversation

shreyaspimpalgaonkar
Copy link
Member

@shreyaspimpalgaonkar shreyaspimpalgaonkar commented Apr 4, 2024

Ellipsis 🚀 This PR description was created by Ellipsis for commit 62ade7e.

Summary:

This PR corrects a typo in the OPENAI_API_KEY environment variable within the error message of the __init__ method in the OpenAIEmbeddingProvider class in the /r2r/embeddings/openai/base.py file.

Key points:

  • Fixed a typo in the error message of the __init__ method in the OpenAIEmbeddingProvider class.
  • The typo was in the name of the environment variable OPENAI_API_KEY.

Generated with ❤️ by ellipsis.dev

Copy link

vercel bot commented Apr 4, 2024

@shreyaspimpalgaonkar is attempting to deploy a commit to the Sciphi-Team Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

👍 Looks good to me!

  • Reviewed the entire pull request up to 62ade7e
  • Looked at 14 lines of code in 1 files
  • Took 22 seconds to review
More info
  • Skipped 0 files when reviewing.
  • Skipped posting 1 additional comments because they didn't meet confidence threshold of 50%.
1. r2r/embeddings/openai/base.py:36:
  • Assessed confidence : 0%
  • Comment:
    The correction of the environment variable name in the error message from 'OPEN_API_KEY' to 'OPENAI_API_KEY' is accurate and consistent with the environment variable check in the code.
  • Reasoning:
    The PR is a simple typo fix. The author has corrected the environment variable name in the error message from 'OPEN_API_KEY' to 'OPENAI_API_KEY'. This is consistent with the environment variable check in the code. No logical, performance, or security bugs are introduced with this change. The change is straightforward and doesn't seem to require any additional context from the codebase.

Workflow ID: wflow_13BsfsBrG6dLQMnU


Not what you expected? You can customize the content of the reviews using rules. Learn more here.

@emrgnt-cmplxty emrgnt-cmplxty merged commit d030668 into SciPhi-AI:main Apr 4, 2024
1 check failed
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.

2 participants