Skip to content

Conversation

ashtul
Copy link
Contributor

@ashtul ashtul commented Mar 1, 2021

Fixes #35

@ashtul ashtul requested review from rafie and oshadmi March 1, 2021 15:25
@codecov
Copy link

codecov bot commented Mar 1, 2021

Codecov Report

Merging #115 (2a980af) into master (8122d94) will decrease coverage by 0.49%.
The diff coverage is 46.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #115      +/-   ##
==========================================
- Coverage   90.17%   89.67%   -0.50%     
==========================================
  Files          12       12              
  Lines        1700     1715      +15     
==========================================
+ Hits         1533     1538       +5     
- Misses        167      177      +10     
Impacted Files Coverage Δ
test/test.py 94.81% <11.11%> (-1.21%) ⬇️
redisearch/query.py 89.63% <100.00%> (-0.88%) ⬇️

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 8122d94...2a980af. Read the comment docs.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 6, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@lgtm-com
Copy link

lgtm-com bot commented Mar 6, 2021

This pull request introduces 1 alert when merging 2a980af into 8122d94 - view on LGTM.com

new alerts:

  • 1 for Variable defined multiple times

@ashtul
Copy link
Contributor Author

ashtul commented Mar 6, 2021

Fixed by #113

@ashtul ashtul closed this Mar 6, 2021
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.

does the client support TFIDF scoring and multisearch?
1 participant