Closed davidvonthenen closed 9 months ago
The updates primarily involve cleaning up and organizing code across the Deepgram project. This includes reordering import statements, removing global usings and adding them locally, correcting minor syntax errors (like missing semicolons), and simplifying method implementations. Additionally, the project's dependencies have been updated, and code analysis tools have been integrated to ensure code quality. These changes contribute to better readability, maintainability, and adherence to best practices within the codebase.
File(s) | Change Summary |
---|---|
Deepgram.Tests/GlobalUsings.cs | Removed global usings for networking and serialization, added them locally. |
Deepgram.Tests/.../ClientTests/*.cs | Reordered import statements, adjusted import order for Deepgram.Models.* . |
Deepgram.Tests/.../ExtensionsTests/HttpClientExtensionTests.cs | Swapped import order of Deepgram.Models.Manage.v1 and Deepgram.Models.Authenticate.v1 . |
Deepgram/Abstractions/AbstractRestClient.cs | Corrected spacing around cancellationToken in SendAsync . |
Deepgram/Constants/Defaults.cs | Removed DEEPGRAM_CONTENT_TYPE constant. |
Deepgram/Deepgram.csproj | Updated dependencies, added Microsoft.CodeAnalysis.NetAnalyzers . |
Deepgram/Encapsulations/HttpClientFactory.cs, Deepgram/Enums/LiveType.cs | Added missing semicolon after namespace declaration. |
Deepgram/LiveClient.cs, Deepgram/ManageClient.cs, Deepgram/OnPremClient.cs, Deepgram/PrerecordedClient.cs | Reordered import statements, adjusted import order for Deepgram.Models.* . |
Deepgram/Logger/LogProvider.cs | Removed unnecessary empty lines in GetLogger . |
Deepgram/Utilities/RequestContentUtil.cs | Simplified CreatePayload , adjusted StringContent constructor call, removed Content-Type setting in CreateStreamPayload . |
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?
These are updates based on the MS .NET Analyzers (linterrs, checks, etc) after enabling them in the project settings. These aren't bad until I look at more automated linters for code formatting.
Summary by CodeRabbit