leona / helix-gpt

Code assistant language server for Helix with support for Copilot/OpenAI/Codeium/Ollama
MIT License
287 stars 19 forks source link

Consider changing an environment var to `OPENAI_API_KEY` #2

Closed David-Else closed 5 months ago

David-Else commented 5 months ago

Hi, thanks for this project!

Please note that the two best and most used CLI apps for ChatGPT, aichat and shell_gpt, use the following to export the API key:

OPENAI_API_KEY=

rather than your:

OPENAI_KEY=123 # required

Changing to use the more commonly accepted format would make things easier as many users will be exporting their key already and making another identical export would be annoying. Cheers!

https://github.com/sigoden/aichat/wiki/Environment-Variables https://github.com/TheR1D/shell_gpt#runtime-configuration-file

EDIT: You might want to go with sgpt DEFAULT_MODEL?

leona commented 5 months ago

Hey thanks for the suggestion! I've made the change for OPENAI_API_KEY. I might leave the model one for the moment, as I plan on supporting copilot and self hosted LLMs, and would like the openai config separate.