Change DG Cloud API from beta to official one#3752
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request correctly updates the Deepgram Cloud API from the beta endpoint to the official one. However, this change leaves behind some dead code related to an unused Deepgram client. My review includes a suggestion to remove this unused code to improve the codebase's maintainability.
| # unused fn | ||
| deepgram_beta = DeepgramClient(os.getenv('DEEPGRAM_API_KEY'), deepgram_beta_options) | ||
| deepgram_beta = DeepgramClient(os.getenv('DEEPGRAM_API_KEY'), deepgram_cloud_options) |
There was a problem hiding this comment.
This deepgram_beta client is marked as unused and doesn't appear to be used anywhere in the codebase. It's good practice to remove dead code to improve maintainability. The variable name is also misleading now that the beta URL is no longer used. Since this client is being removed, the deepgram_cloud_options variable and its setup on lines 249-250 and 258 also become unused and can be removed.
beastoin
deleted the
change-deepgram-cloud-api-from-beta-to-official-one
branch
|
lgtm @thainguyensunya |
2 participants
Changes: