flows-network / telegram-llm

A Telegram LLM bot written in Rust
https://flows.network/
GNU General Public License v3.0
66 stars 21 forks source link

Remove OpenAI key name and use the default key #3

Closed juntao closed 1 year ago

juntao commented 1 year ago

Hello, I am a code review bot on flows.network. Here are my reviews of code commits in this PR.


Commit f05cfa337ad370b3828e39b53aafcc94c3e54a4a

The patch removes the use of a custom OpenAI key and instead uses the default key for chat completion. It also updates the function call from chat_completion to chat_completion_default_key to reflect this change.

There do not appear to be any immediate problems with this patch. However, it is possible that the custom OpenAI key was being used for a specific reason, and the change to the default key could impact the behavior of the program. It would be important to ensure that all necessary testing is done to confirm that the change does not cause any regressions or errors. Additionally, depending on the context in which this code is being used, it may be necessary to ensure that the use of the default OpenAI key does not violate any terms of service or security requirements.

alabulei1 commented 1 year ago

Thank you!