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.

Sign up for GitHub

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

Jump to bottom

Support FT.SYNADD FT.SYNUPDATE FT.SYNDUMP commands #55

Merged
merged 2 commits into from
Apr 10, 2020
Merged

Support FT.SYNADD FT.SYNUPDATE FT.SYNDUMP commands #55

merged 2 commits into from
Apr 10, 2020

Conversation

dengliming
Copy link
Contributor

Solved #42

@codecov
Copy link

codecov bot commented Apr 10, 2020 •
edited
Loading

Codecov Report

Merging #55 into master will increase coverage by 0.15%.
The diff coverage is 78.57%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #55      +/-   ##
==========================================
+ Coverage   72.59%   72.74%   +0.15%     
==========================================
  Files          12       12              
  Lines        1080     1108      +28     
==========================================
+ Hits          784      806      +22     
- Misses        236      239       +3     
- Partials       60       63       +3
Impacted Files Coverage Δ
redisearch/client.go 79.66% <78.57%> (-0.10%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a595a0d...323b9f2. Read the comment docs.

@filipecosta90 filipecosta90 self-requested a review April 10, 2020 15:56
filipecosta90
filipecosta90 approved these changes Apr 10, 2020
View reviewed changes
Copy link
Collaborator

@filipecosta90 filipecosta90 left a comment

Choose a reason for hiding this comment

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

Thanks @dengliming . Merging 👍

@filipecosta90 filipecosta90 merged commit 76b1c66 into RediSearch:master Apr 10, 2020
@filipecosta90 filipecosta90 mentioned this pull request Apr 10, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@filipecosta90 filipecosta90 filipecosta90 approved these changes

Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dengliming @filipecosta90