Skip to content

Add openairc file#41

Closed
tom-doerr wants to merge 1 commit into
openai:mainfrom
tom-doerr:add_openairc
Closed

Add openairc file#41
tom-doerr wants to merge 1 commit into
openai:mainfrom
tom-doerr:add_openairc

Conversation

@tom-doerr

Copy link
Copy Markdown

After merging this PR, users will be able to store their authentication information in the openairc file.

Currently, if the credentials are not in the environment, every application has to load the credentials itself, e.g.

Having the openai module load the config means less work for users and developers.

~/.config/openairc:

[api]
Organization = org-j----------------------P
APIKey = sk-U----------------------------------------------A

@hallacy hallacy left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR! I like the idea of a config file. Any chance you could clear up those merge conflicts and we can get this in?

@hallacy

hallacy commented Jan 26, 2022

Copy link
Copy Markdown
Collaborator

Actually, I think this feature was mostly added with the addition of the api_key_path feature (#42). Any chance you could merge your feature with that?

@hallacy

hallacy commented Feb 1, 2022

Copy link
Copy Markdown
Collaborator

Gonna close this for now. Feel free to reopen!

@hallacy hallacy closed this Feb 1, 2022
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

Successfully merging this pull request may close these issues.

2 participants