Skip to content

API documentation of EXPIRETIME and PEXPIRETIME commands#1582

Merged
oranagra merged 9 commits intoredis:masterfrom
ny0312:expiretime-pexpiretime-doc
Jun 15, 2021
Merged

API documentation of EXPIRETIME and PEXPIRETIME commands#1582
oranagra merged 9 commits intoredis:masterfrom
ny0312:expiretime-pexpiretime-doc

Conversation

@ny0312
Copy link
Contributor

@ny0312 ny0312 commented May 31, 2021

Add API documentation for EXPIRETIME and PEXPIRETIME commands introduced in this PR: redis/redis#8474

@oranagra oranagra added the to-be-merged should probably be merged soon label Jun 1, 2021
Copy link
Contributor

@zuiderkwast zuiderkwast left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good in general. A few nits.

ny0312 and others added 7 commits June 1, 2021 13:05
Co-authored-by: Viktor Söderqvist <[email protected]>
Co-authored-by: Viktor Söderqvist <[email protected]>
Co-authored-by: Viktor Söderqvist <[email protected]>
Co-authored-by: Viktor Söderqvist <[email protected]>
Co-authored-by: Viktor Söderqvist <[email protected]>
Co-authored-by: Viktor Söderqvist <[email protected]>
Co-authored-by: Viktor Söderqvist <[email protected]>
Copy link
Contributor

@madolson madolson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@yoav-steinberg yoav-steinberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor English fix.

Co-authored-by: yoav-steinberg <[email protected]>
@oranagra oranagra merged commit 586b105 into redis:master Jun 15, 2021
@ny0312 ny0312 deleted the expiretime-pexpiretime-doc branch July 2, 2021 17:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

to-be-merged should probably be merged soon

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants