Closed davidvonthenen closed 8 months ago
This comprehensive update focuses on refining and enhancing the Deepgram .NET SDK across various client tests, utility classes, and client implementations. It includes method signature updates, improved exception handling, refined test setups, and the introduction of new functionalities such as custom headers and cancellation token support. Additionally, it addresses HTTP client configuration, timeout settings, and the organization of constants and schemas within the SDK, aiming to improve flexibility, readability, and maintainability.
File Paths | Change Summaries |
---|---|
Deepgram.Tests/UnitTests/ClientTests/... |
Adjusted test methods for exception handling, method signatures, mock setups, and added clarity in comments. |
Deepgram/Abstractions/... , Deepgram/Clients/... |
Introduced new methods with additional parameters for headers and cancellation tokens, updated method calls, and refined URI construction. |
Deepgram/Models/... |
Added NoopSchema.cs files across various namespaces as placeholders or base schemas. |
Deepgram/Utilities/QueryParameterUtil.cs , Deepgram/Abstractions/Utilities.cs |
Refactored methods for URL formatting and renamed classes for clarity. |
Deepgram/Deepgram.csproj , examples/... |
Updated project configurations and target frameworks. |
Deepgram/Factory/HttpClientFactory.cs |
Enhanced HTTP client configuration with infinite timeout and protocol check. |
metadata
field, the extensive updates and refactoring could facilitate such a feature in future updates.HttpClientWrapper
and mocking HTTP calls.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?
Updated PR based on code rabbit feedback.
Fixes issue: https://github.com/deepgram/deepgram-dotnet-sdk/issues/234
This implements:
Summary by CodeRabbit
headers
parameter to various client methods for custom request headers.NoopSchema
classes across different models for future extensibility.RequestContentUtil
toHttpRequestUtil
for clarity.//Arrange
to// Input and Output
in test descriptions for enhanced clarity.