Skip to content

Fix issue where OpenAI API key was not used#31

Merged
guywaldman merged 6 commits intomainfrom
user/guywaldman/openai-fix
Jul 23, 2024
Merged

Fix issue where OpenAI API key was not used#31
guywaldman merged 6 commits intomainfrom
user/guywaldman/openai-fix

Conversation

@guywaldman
Copy link
Owner

@guywaldman guywaldman commented Jul 23, 2024

This PR fixes an issue where the OpenAI API key defined in the configuration was not used. I believe this regression was introduced in #27.

Detailed Changes

  • Added back the Open AI key from the configuration in mcli_config.rs
  • Added a regression test
  • Changed logs in tests so that the API key won't be printed

Resolves #29

@guywaldman guywaldman changed the title Fix issue where Open AI key was not used Fix issue where OpenAI API key was not used Jul 23, 2024
@guywaldman guywaldman merged commit 77982bc into main Jul 23, 2024
@guywaldman guywaldman deleted the user/guywaldman/openai-fix branch July 23, 2024 04:59
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.

API Key not set

1 participant