Skip to content

Conversation

alphabetkrish
Copy link
Contributor

@alphabetkrish alphabetkrish commented Oct 28, 2022

Description

fix #2231

Describe your pull request here


Checklist

  • Does npm test pass with this change (including linting)?
  • Is the new or changed code fully tested?
  • Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?

@leibale leibale requested a review from simonprickett October 28, 2022 22:44
Copy link
Contributor

@simonprickett simonprickett left a comment

Choose a reason for hiding this comment

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

Made some inline suggestions cc @leibale

@simonprickett simonprickett self-requested a review November 4, 2022 13:19
@leibale leibale merged commit e0e96ae into redis:master Nov 6, 2022
florian-schunk pushed a commit to florian-schunk/node-redis that referenced this pull request Jun 18, 2025
* fix(doc): created doc for using timeseries

* fix(doc): created doc for using timeseries

* Apply suggestions from code review

Co-authored-by: Simon Prickett <[email protected]>

Co-authored-by: Simon Prickett <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Extend TimeSeries README.md
3 participants