Kusto has TODOs in the code
Open
Nobody has claimed this yet.
server-Azure.Mcp
tools-Kusto
- Dominant language
- C#
- Stars
- 3.7k
- Forks
- 624
- Avg merge
- 2d 20h
- Merged PRs (30d)
- 220
Description
Review these TODOs in the Kusto light client that were deleted for a clean [pr](https://github.com/Azure/azure-mcp/pull/368). If unnecessary, feel free to close this issue.
1. In KustoResult: a todo comment was there to populate more properties in the KustoResult model and return it from FromHttpResponseMessage method.
2. In KustoClient: Add “ClientRequestId” as a header (was a todo comment in in GenerateRequestMethod).
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.