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

Fallback to openai #58

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

Alby11
Copy link

@Alby11 Alby11 commented Dec 3, 2024

This branch has been created to implement defaults for missing data in config file and use environment variables for api key. If the config file is missing, OpenAI will be used as default service.

@Alby11 Alby11 closed this Dec 3, 2024
@Alby11 Alby11 reopened this Dec 3, 2024
@Alby11
Copy link
Author

Alby11 commented Dec 3, 2024

Dear Tom,

I am currently conducting thorough testing on this branch and would appreciate your feedback. Thank you for the opportunity to contribute to your outstanding project.

Best regards,
Alberto

@tom-doerr
Copy link
Owner

Didn't see any issues, but it's unlikely I would have spotted them if there are any just from looking at the diff. I asked Copilot and it doesn't see any issues as well so far
Screenshot_20241204_114535_Samsung Internet

@tom-doerr
Copy link
Owner

Do you think it's ready for merging?

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