Closed davidvonthenen closed 9 months ago
The recent updates focus on simplifying method names across various classes by removing redundant prefixes related to projects and keys. This change aims to enhance clarity and consistency in naming, making the codebase more intuitive to navigate and understand. The modifications span across unit tests, logging functionalities, and client management, reflecting a streamlined approach to accessing and managing project-related information such as keys, invites, members, usage, and balances.
File(s) | Summary |
---|---|
Deepgram.Tests/.../ManageClientTest.cs |
Renamed functions by removing "Project" and "Key" prefixes for simplicity. Updated test methods accordingly. |
Deepgram/Logger/Log.cs |
Renamed CreateProjectKeyError to CreateKeyError for clarity. |
Deepgram/ManageClient.cs |
Updated method names by removing "Project" and streamlining terms (e.g., GetProjectKeys to GetKeys , CreateProjectKey to CreateKey ). This includes changes for keys, invites, members, usage, and balances. |
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?
It looks like the Manage function names also needed to be changed to the more minimalist form. This matches the recent changes with the model names in this PR https://github.com/deepgram/deepgram-dotnet-sdk/pull/190. Effectively, this removes "GetProject" from every function name (ie.
GetProjectUsageRequest()
->GetUsageRequest()
) to make it easier to call functions.Summary by CodeRabbit