Closed davidvonthenen closed 1 month 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 20 minutes and 0 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 dff557d3f5e4a3c80764c6001f03c09796af4721 and 6811cd2d86f2480bf254fa3fec6441cccf406707.
The pull request introduces several modifications to the Deepgram.Dev.sln
solution file, including the addition of two new projects: reconnect_same_object
and its corresponding test project ReconnectStreaming
. Additionally, the instantiation logic for the ListenWebSocketClient
in the Program.cs
file has been updated to utilize a factory method. The Program.cs
in the reconnect_same_object
tests has undergone significant restructuring to support multiple reconnections to the WebSocket client. Lastly, the ReconnectStreaming.csproj
file has been updated to include a reference to the Deepgram.Microphone
project.
File Path | Change Summary |
---|---|
Deepgram.Dev.sln | Added projects: reconnect_same_object and ReconnectStreaming . Updated global sections to include these projects. |
examples/speech-to-text/websocket/microphone/Program.cs | Updated instantiation of ListenWebSocketClient to use ClientFactory.CreateListenWebSocketClient . |
tests/edge_cases/reconnect_same_object/Program.cs | Restructured connection logic to support multiple WebSocket client instances in a loop. Updated instantiation to use ClientFactory.CreateListenWebSocketClient . |
tests/edge_cases/reconnect_same_object/ReconnectStreaming.csproj | Added project reference to Deepgram.Microphone . |
WebSocketMessage
class constructor and property directly relate to the handling of WebSocket messages, which may be relevant for the new ReconnectStreaming
project that involves WebSocket connections.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?
Proposed changes
This updates the Listen WS example to add more color to what is going on. It was also missing from the Developer Solution file for some reason.
Types of changes
What types of changes does your code introduce to the community .NET SDK? Put an
x
in the boxes that applyChecklist
Put an
x
in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.Further comments
NA
Summary by CodeRabbit
New Features
reconnect_same_object
.ReconnectStreaming
.Improvements
Bug Fixes