Update for TopK.Add which returns expelled items

[ashtul]

No description provided.

[codecov-io]

Codecov Report

Merging #3 into master will decrease coverage by 0.29%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master       #3     +/-   ##
=========================================
- Coverage   98.79%   98.49%   -0.3%     
=========================================
  Files           2        2             
  Lines         331      333      +2     
=========================================
+ Hits          327      328      +1     
- Misses          4        5      +1

Impacted Files	Coverage Δ
test_commands.py	100% <100%> (ø)	⬆️
redisbloom/client.py	97.47% <100%> (-0.49%)	⬇️

---

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 231bc91...4f8ac00. Read the comment docs.

[ashtul]

If anything changes in the future. They can be eliminated. I debated it myself.

…

On 21 Jun 2019, at 12:08, Guy Korland ***@***.***> wrote: @gkorland commented on this pull request. In redisbloom/client.py: > @@ -101,34 +104,35 @@ def __init__(self, *args, **kwargs): self.BF_RESERVE : bool_ok, #self.BF_ADD : spaceHolder, #self.BF_MADD : spaceHolder, - self.BF_INSERT : spaceHolder, - self.BF_EXISTS : spaceHolder, - self.BF_MEXISTS : spaceHolder, - self.BF_SCANDUMP : spaceHolder, - self.BF_LOADCHUNK : spaceHolder, + #self.BF_INSERT : spaceHolder, + #self.BF_EXISTS : spaceHolder, @ashtul why do we keep the commented out lines? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.