-
Notifications
You must be signed in to change notification settings - Fork 44.6k
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
❇️ Improved OpenAI API Key Insert to Env #2486
❇️ Improved OpenAI API Key Insert to Env #2486
Conversation
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request. |
Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly. |
This is a mass message from the AutoGPT core team. For more details (and for infor on joining our Discord), please refer to: |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #2486 +/- ##
==========================================
- Coverage 70.87% 70.68% -0.20%
==========================================
Files 68 68
Lines 3317 3326 +9
Branches 531 532 +1
==========================================
Hits 2351 2351
- Misses 788 798 +10
+ Partials 178 177 -1
☔ View full report in Codecov by Sentry. |
Are we ok with putting this in place temporarily? It's probably worth waiting for #4604 to be implemented so that we don't change things for users too often. I'll flag it for discussion. |
✅ Deploy Preview for auto-gpt-docs canceled.
|
You changed AutoGPT's behaviour. The cassettes have been updated and will be merged to the submodule when this Pull Request gets merged. |
Background
Relates to #2148 but most commonly to fix a lot of users trying to use the AutoGPT and failed to set up their
.env
file or to set up the environment variable properly.Changes
Improved
check_openai_api_key()
functionDocumentation
Documented in the function itself. It's a small singular atomic function that handles checking and having the user input into the environment variable and
Config
of AutoGPT.Test Plan
Manually run the function. Unable to write a unit test script that can handle user input.
PR Quality Checklist