Skip to content
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

fix(#2231): created doc for using timeseries #2312

Merged
merged 3 commits into from
Nov 6, 2022
Merged

fix(#2231): created doc for using timeseries #2312

merged 3 commits into from
Nov 6, 2022

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)?

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

packages/time-series/README.md Outdated Show resolved Hide resolved
packages/time-series/README.md Outdated Show resolved Hide resolved
packages/time-series/README.md Outdated Show resolved Hide resolved
packages/time-series/README.md Outdated Show resolved Hide resolved
packages/time-series/README.md Outdated Show resolved Hide resolved
packages/time-series/README.md Outdated Show resolved Hide resolved
packages/time-series/README.md Outdated Show resolved Hide resolved
packages/time-series/README.md Outdated Show resolved Hide resolved
packages/time-series/README.md Outdated Show resolved Hide resolved
packages/time-series/README.md Outdated Show resolved Hide resolved
Co-authored-by: Simon Prickett <simon@crudworks.org>
@simonprickett simonprickett self-requested a review November 4, 2022 13:19
@leibale leibale merged commit e0e96ae into redis:master Nov 6, 2022
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