-
Notifications
You must be signed in to change notification settings - Fork 8
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
Add delete feature #102
Add delete feature #102
Conversation
Add simple get function Add tests Co-authored-by: Ahlberg-iths <113943951+Ahlberg-iths@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
God job!
Seems to work as intended :)
Co-authored-by: Ahlberg-iths <113943951+Ahlberg-iths@users.noreply.github.com>
Co-authored-by: Ahlberg-iths <113943951+Ahlberg-iths@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I dont think the get method is correct.
If the key exists it should just return the key.
https://redis.io/commands/get/
im assuming this is the same get as #8
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know @safstromo , it seems to be correct to me https://redis.io/commands/get/
See where it says Return: Bulk string reply
👍 |
43220a7
Kudos, SonarCloud Quality Gate passed! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me :)
Add delete function, simple get function and tests