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

Add common environment variables #8

Closed
dustinblackman opened this issue Dec 1, 2023 · 2 comments
Closed

Add common environment variables #8

dustinblackman opened this issue Dec 1, 2023 · 2 comments

Comments

@dustinblackman
Copy link
Owner

Overview

Following up on #6, add common environment variables shared across other apps for OpenAI such as:

  • OPENAI_MODEL
  • OPENAI_API_KEY
  • OPENAI_KEY
@suhr
Copy link

suhr commented Dec 23, 2023

It would be nice to support OLLAMA_URL as well.

@dustinblackman
Copy link
Owner Author

While I see the appeal of this, Oatmeal's environment variable configs are tied to clap, which only allows one environment variable. Decoupling that and having to mange several variants of envs sounds tedious. The following I'd say is good enough.

export OPENAI_KEY=keysecretkey
export OATMEAL_OPENAI_TOKEN=$OPENAI_KEY

Marking as wont fix.

@dustinblackman dustinblackman closed this as not planned Won't fix, can't repro, duplicate, stale Jan 31, 2024
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

2 participants