Undeprecate cluster slots command

[hpatro]

Undeprecate cluster slots command. This command is widely used by clients to form the cluster topology and with the recent change to improve performance of CLUSTER SLOTS command via #53 as well as us looking to further improve the usability via #517, it makes sense to undeprecate this command.

[hpatro]

@valkey-io/core-team please take a look.

Should we update the history as well to indicate we are un-deprecating the command ?

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.15%. Comparing base (c478206) to head (33188e3).
Report is 2 commits behind head on unstable.

Additional details and impacted files

@@             Coverage Diff              @@
##           unstable     #536      +/-   ##
============================================
+ Coverage     69.71%   70.15%   +0.43%     
============================================
  Files           109      109              
  Lines         61864    59904    -1960     
============================================
- Hits          43131    42027    -1104     
+ Misses        18733    17877     -856     

Files	Coverage Δ
src/commands.def	100.00% <ø> (ø)

... and 86 files with indirect coverage changes

[madolson]

You need to regenerate commands.def .

[madolson]

Should we update the history as well to indicate we are un-deprecating the command ?

No, history is for command argument changes. We can just un-deprecate it.

[madolson]

I'm going to be more explicit and ask for a vote from @valkey-io/core-team.

[enjoy-binbin]

LGTM.

[zuiderkwast]

We should still mention the drawbacks in the docs and recommend cluster shards for large and very scattered slot distributions.