Skip to content

fix: zrange typing#51

Open
YDrogen wants to merge 1 commit intovalkey-io:mainfrom
YDrogen:main
Open

fix: zrange typing#51
YDrogen wants to merge 1 commit intovalkey-io:mainfrom
YDrogen:main

Conversation

@YDrogen
Copy link
Copy Markdown

@YDrogen YDrogen commented Sep 2, 2025

Added number as possible type for stop parameter in zrange functions
Fixes #49

Signed-off-by: YDrogen <ydrogen45@gmail.com>
@joe-re
Copy link
Copy Markdown

joe-re commented Jan 21, 2026

@mcollina Could you please review this PR when you have a chance?

This fixes the type inconsistency where zrange's stop parameter doesn't accept number while start does, which requires users to use unnecessary workarounds like String(stop).

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.

Erroneous documentation or type definition

2 participants