Closed davidvonthenen closed 4 months ago
[!WARNING]
Rate limit exceeded
@dvonthenen has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 6 minutes and 37 seconds before requesting another review.
How to resolve this issue?
After the wait time has elapsed, a review can be triggered using the `@coderabbitai review` command as a PR comment. Alternatively, push new commits to this PR. We recommend that you space out your commits to avoid hitting the rate limit.How do rate limits work?
CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our [FAQ](https://coderabbit.ai/docs/faq) for further information.Commits
Files that changed from the base of the PR and between 4e806420d97509b0d58d425a95dd81fa088919b0 and 1b1d97c90fd0f0e81c36997a54c7910cdef9240f.
The overall change involves replacing the previous usage of PreRecordedClient
and LiveClient
with ClientFactory.CreateListenRESTClient()
and ClientFactory.CreateListenWebSocketClient()
, respectively, across different Program.cs
files in test directories. Import statements have also been updated accordingly to reflect the usage of Deepgram.Models.Listen.v1.REST
and Deepgram.Models.Listen.v1.WebSocket
instead of their older counterparts.
Files | Change Summary |
---|---|
tests/expected_failures/rest/bad_host/... |
Replaced PreRecordedClient with ClientFactory.CreateListenRESTClient() and updated import statements from Deepgram.Models.PreRecorded.v1 to Deepgram.Models.Listen.v1.REST . |
tests/expected_failures/rest/throw_exception/... |
Replaced PreRecordedClient with ClientFactory.CreateListenRESTClient() and updated import statements from Deepgram.Models.PreRecorded.v1 to Deepgram.Models.Listen.v1.REST . |
tests/expected_failures/rest/trigger_cancel/... |
Modified import statements to use Deepgram.Models.Listen.v1.REST and updated client instantiation to ClientFactory.CreateListenRESTClient() . |
tests/expected_failures/websocket/exercise_timeout/... |
Changed from using LiveClient to ClientFactory.CreateListenWebSocketClient() for WebSocket communication; adjusted event subscription and client disposal accordingly. |
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
This updates all examples to migrate from the legacy/deprecated namespaces to move to the new ones. The previous PR shows all the examples work using the deprecated namespaces and this one shows they work using the latest namespaces.
Adds missing
ClientFactory
functionCreateListenRESTClient
.Tested all examples, works as expected. Checked each project for deprecation notices and to make sure namespaces are correct.
Summary by CodeRabbit
PreRecordedClient
andLiveClient
with newClientFactory
methods for REST and WebSocket communication.Deepgram.Models.Listen.v1
namespaces for REST and WebSocket.These changes modernize the communication client setup and streamline the codebase for future updates.