kvrocks icon indicating copy to clipboard operation
kvrocks copied to clipboard

Tracking issues: add the support of all missing commands in the list type

Open infdahai opened this issue 1 year ago • 8 comments

Search before asking

  • [X] I had searched in the issues and found no similar issues.

Motivation

This issue considers to support the blmove, blmpop, brpoplpush, lmpop, lpos commands.

Solution

  • [x] #1521
  • [x] #1522
  • [ ] #1523
  • [x] #1526
  • [x] #1528

Are you willing to submit a PR?

  • [ ] I'm willing to submit a PR!

infdahai avatar Jun 22 '23 08:06 infdahai

i can do this

IoCing avatar Jun 26 '23 08:06 IoCing

It will be better if you can open an issue for every command you mention here.

PragmaTwice avatar Jun 27 '23 08:06 PragmaTwice

It will be better if you can open an issue for every command you mention here.

OK, I will open some issues in the evening.

infdahai avatar Jun 27 '23 08:06 infdahai

Hi @IoCing, you can select specific commands if you want.

infdahai avatar Jun 27 '23 14:06 infdahai

@infdahai Please have a look, I see that the LLEN, LINSERT, LPOP, RPUSH are also supported. The formatting in the file is a little weird. image

torwig avatar Jun 27 '23 14:06 torwig

@infdahai Please have a look, I see that the LLEN, LINSERT, LPOP, RPUSH are also supported. The formatting in the file is a little weird. image

OK, I'm sorry for this. My window wasn't big enough, so I didn't notice the layout. I update this issue now.

infdahai avatar Jun 27 '23 14:06 infdahai

@infdahai And you can edit this source file to list each command on a separate line, introducing a small PR.

torwig avatar Jun 27 '23 14:06 torwig

@infdahai And you can edit this source file to list each command on a separate line, introducing a small PR.

I get it.

infdahai avatar Jun 27 '23 14:06 infdahai