redis-doc
redis-doc copied to clipboard
Command Does Not Include Response Type
commands.json
should include an expected response type for each command. For example, INCR
should have a response
property of "integer"
, because under normal operating conditions it responds with a long integer.
Hello @skeggse
I'm not sure that I understand the issue - commands.json at its current form does not feature the response type as it is already specified in each command's .md file.
I opened this six years ago, and have no recollection about the problem I was trying to solve or any additional details. I'll consider this closed.
Sure - my apologies for the belated reply.
@itamarhaber, if I recall correctly, my use case was having response types in the JSON so that machines could process them. Specifically, there was a TypeScript library that derived typings from this project so that types could be verified via static analysis at compile time.
@nl-brett-stime === @skeggse ?
Ok, I can see how this could be handy for clients that want to know beforehand about the response's type. Funnily enough, with RESP3 this should be a moot point. Still, it may be worth the effort if only for completeness' sake.
Reopening this for further consideration
Yeah this seems reasonable.
@nl-brett-stime === @skeggse ?
Nope!
Closing as stale.