Skip to content

Conversation

@devinsburke
Copy link
Contributor

Added support for SLOP. Changed the int to an *int pointer (nullable) because SLOP 0 is a valid activity. The existing slop int attribute of the query struct was completely unused/unreferenced previously.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 2, 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
No Duplication information No Duplication information

@codecov
Copy link

codecov bot commented Jan 2, 2021

Codecov Report

Merging #97 (d2e39e9) into master (f79df23) will decrease coverage by 0.13%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #97      +/-   ##
==========================================
- Coverage   76.77%   76.63%   -0.14%     
==========================================
  Files          13       13              
  Lines        1098     1100       +2     
==========================================
  Hits          843      843              
- Misses        198      199       +1     
- Partials       57       58       +1     
Impacted Files Coverage Δ
redisearch/query.go 85.71% <0.00%> (-1.19%) ⬇️

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 f79df23...d2e39e9. Read the comment docs.

@filipecosta90
Copy link
Collaborator

@devinsburke can we add a test for it?

@filipecosta90 filipecosta90 self-requested a review April 15, 2021 10:55
@filipecosta90 filipecosta90 added the state:needs-test-pr Requires further testing to be merged label Apr 15, 2021
@sonarqubecloud
Copy link

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

@filipecosta90 filipecosta90 merged commit 75f19ff into RediSearch:master Apr 22, 2021
@filipecosta90
Copy link
Collaborator

Merged @devinsburke . Thank you!
I'll add some extra testing afterwards...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

state:needs-test-pr Requires further testing to be merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants