Skip to content
This repository has been archived by the owner on Feb 21, 2023. It is now read-only.

Added ZREVRANGEBYLEX command #201

Merged
merged 3 commits into from
Mar 3, 2017
Merged

Added ZREVRANGEBYLEX command #201

merged 3 commits into from
Mar 3, 2017

Conversation

marijngiesen
Copy link
Contributor

Added the ZREVRANGEBYLEX command and tests. Closes #200.

@popravich
Copy link
Contributor

Good catch, thanks.
Please fix flake8 errors (run make flake to see the list)

@codecov-io
Copy link

codecov-io commented Mar 3, 2017

Codecov Report

Merging #201 into master will decrease coverage by -0.1%.
The diff coverage is 98.33%.

@@            Coverage Diff            @@
##           master     #201     +/-   ##
=========================================
- Coverage   95.42%   95.33%   -0.1%     
=========================================
  Files          55       55             
  Lines        7173     7227     +54     
  Branches      584      592      +8     
=========================================
+ Hits         6845     6890     +45     
- Misses        239      244      +5     
- Partials       89       93      +4
Impacted Files Coverage Δ
aioredis/commands/sorted_set.py 100% <100%> (ø)
tests/sorted_set_commands_test.py 99.58% <97.5%> (+0.03%)
aioredis/commands/transaction.py 82.63% <0%> (-2.09%)
aioredis/pubsub.py 93.12% <0%> (-1.06%)
tests/conftest.py 86.96% <0%> (-1.01%)
aioredis/sentinel/pool.py 79.37% <0%> (-0.78%)
aioredis/commands/server.py 76.59% <0%> (+1.41%)

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 bc44c9a...431146e. Read the comment docs.

@marijngiesen
Copy link
Contributor Author

The flake8 errors have been fixed

@popravich popravich merged commit 2653a2d into aio-libs-abandoned:master Mar 3, 2017
@popravich
Copy link
Contributor

Thanks

@marijngiesen marijngiesen deleted the zrevrangebylex branch March 3, 2017 09:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants