HRANDFIELD feature by slorello89 · Pull Request #2090 · StackExchange/StackExchange.Redis
Navigation Menu
{{ message }}
StackExchange / StackExchange.Redis Public
- Notifications You must be signed in to change notification settings
- Fork 1.6k
Merged
Conversation
Copy link Copy Markdown
Collaborator
slorello89
commented
Apr 13, 2022
slorello89
commented
Implements HRANDFIELD as a part of #2055.
slorello89
requested a review
from NickCraver
slorello89
mentioned this pull request
slorello89 and others added 4 commits
April 13, 2022 17:26NickCraver approved these changes Apr 14, 2022
Copy link Copy Markdown
Collaborator
NickCraver
left a comment
NickCraver
left a comment
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tweaked some docs and formatting - this was super close already, thanks!!
| /// </summary> | ||
| /// <param name="key">The key of the hash.</param> | ||
| /// <param name="flags">The flags to use for this operation.</param> | ||
| /// <returns>A random hash field name or <see cref="RedisValue.Null"/> if the hash is empty.</returns> |
Copy link Copy Markdown
Collaborator
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: can a hash be empty? Or should we say the hash does not exist?
NickCraver
merged commit
6b37116
into
main
NickCraver
deleted the
feature/hrandmember
branch
NickCraver
added
➕ enhancement
⚙️ area:commands
⚙️ area:API
labels
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment