Aditya Chowdhary

Inconsistent `LPUSH`: Command returns OK instead of Integer(length of list)

4

## Steps to reproduce ```shell 127.0.0.1:7379> lpush mylist "test" ``` ## Expected output The expected output when the above set of commands (maybe when run on Redis) ```shell 127.0.0.1:6379> LPUSH...

Updates documentation for LPUSH command as per #835

#972: fix: Fixes output of RPUSH command

5

Fixes #972 by returning length instead of OK in RPUSH.

Inconsistent `RPUSH`: Command returns OK instead of Integer(length of list)

2

## Steps to reproduce ``` 127.0.0.1:7379> RPUSH mylist 1000 2000 ``` ## Expected output The expected output when the above set of commands (maybe when run on Redis) ``` 127.0.0.1:6379>...

Fixes #933 by returning the length instead of OK. Minor side change - Integration test for RPUSH used LPUSH command like below, which was fixed https://github.com/DiceDB/dice/blob/887d693113e266c8525c75b103346ced5fbdf997/integration_tests/commands/async/deque_test.go#L124 Would also like to...

fixes #1019 Edit: Now ready for review Currently draft PR as updating integration tests is remaining. Remaining tasks are completed.

Learnt through painful experience that it was important to fetch upstream before attempting to merge with upstream/master.