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

rebrand OPENAI_API_KEY to something generic if using litellm models #651

Open
lucasliet opened this issue Dec 6, 2024 · 0 comments
Open

Comments

@lucasliet
Copy link

lucasliet commented Dec 6, 2024

i've made this PR #575 to make it work with Gemini models
it turns out, it was already working on main branch, but you have to set the gemini api key as "OPENAI_API_KEY" to make it work, which could lead to confusion

maybe the key could be more generic, or have more than one key
and use OPENAI_API_KEY only if litellm is not enable, otherwise use the generic one

and reference it on README.md usage

also had to inject google-generativeai
pipx inject shell_gpt google-generativeai

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

No branches or pull requests

1 participant