Skip to content

docs: update .NET SCAN command status to Done - #18

Closed
Aryex wants to merge 2 commits into
mainfrom
alexl/agent/csharp-scan-refactor
Closed

docs: update .NET SCAN command status to Done#18
Aryex wants to merge 2 commits into
mainfrom
alexl/agent/csharp-scan-refactor

Conversation

@Aryex

@Aryex Aryex commented Jun 4, 2026

Copy link
Copy Markdown
Owner

Summary

Update .NET SCAN command status from "Not started" to "Done" in the commands progress table.

Source Commits

  • 3f07489 — Scan Command Refactor (#316)

Changes

  • Updated .NET column for scan from "Not started" to "Done" in the commands implementation progress table

Context

PR valkey-io/valkey-glide-csharp#316 refactors all SCAN commands in the C# client to use a unified IAsyncEnumerable pattern. Key changes:

  • ScanAsync moved from shared IGenericCommands/IGenericClusterCommands to client-specific IGlideClient and IGlideClusterClient interfaces, now returning IAsyncEnumerable<ValkeyKey> instead of cursor-based tuples
  • SetScanAsync and SortedSetScanAsync moved to IBaseClient partial interfaces with simplified ScanOptions? parameter replacing individual pattern/pageSize/cursor/pageOffset parameters
  • ClusterScanCursor made internal (implementation detail)
  • Batch APIs (SetScan, SortedSetScan) aligned to use ScanOptions with consistent parameter ordering

The cluster scan how-to page already has correct C# examples using the new API. The sscan and zscan entries were already marked "Done" for .NET.

cc @alexr-bq


This PR was generated by the doc-trigger pipeline.

hashrr and others added 2 commits June 4, 2026 11:13
…key-io#222)

Signed-off-by: hashrr <harsha7697@gmail.com>
Signed-off-by: Harshavardhan Gowda <harsha7697@gmail.com>
Signed-off-by: Alex Le <alex.le@improving.com>
Co-authored-by: Alex Le <alex.le@improving.com>
@Aryex Aryex closed this Jun 4, 2026
@Aryex
Aryex deleted the alexl/agent/csharp-scan-refactor branch June 4, 2026 20:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants