Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Quoting is now necessary, but shouldn't be #23

Closed
dwdougherty opened this issue Jan 15, 2025 · 4 comments
Closed

Quoting is now necessary, but shouldn't be #23

dwdougherty opened this issue Jan 15, 2025 · 4 comments

Comments

@dwdougherty
Copy link

dwdougherty commented Jan 15, 2025

On the INCRBYFLOAT command page, the interactive CLI widget is now throwing errors:

redis> SET mykey 10.50
(error) syntax error
redis> INCRBYFLOAT mykey 0.1
(error) ERR wrong number of arguments for command
redis> INCRBYFLOAT mykey -5
(error) ERR wrong number of arguments for command
redis> SET mykey 5.0e3
(error) syntax error
redis> INCRBYFLOAT mykey 2.0e2
(error) ERR wrong number of arguments for command
redis> 

Quoting the numbers works, but this shouldn't be necessary (still works fine in redis-cli). Since these errors just started appearing, they may be because of the most recent changes.

Same issue on the HINCRBY page. There may be lots of others.

Cc: @paoloredis

@paoloredis
Copy link

paoloredis commented Jan 15, 2025

cc: @itamarhaber

@dwdougherty
Copy link
Author

@paoloredis reverted to the previous version, which has the BITPOS issues/weirdness, but at least it's not breaking all over the place. :)

@dwdougherty
Copy link
Author

But, now there are a raft of other issues. See:

LMOVE
GEOADD

Not sure what's going on, but it needs to be fixed.

@paoloredis
Copy link

paoloredis commented Jan 22, 2025

LMOVE, GEOADD etc. have been fixed, but the BITPOS issue persists.

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

No branches or pull requests

2 participants