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.

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

scan command implementation #329 #334

Merged
merged 5 commits into from
Nov 15, 2017
Merged

scan command implementation #329 #334

merged 5 commits into from
Nov 15, 2017

Conversation

DrMegavolt
Copy link
Contributor

This is implementation for scan command.
scan-command.helper allow all kind *SCAN command
if this is ok, I can add them as well

@stipsan
Copy link
Owner

stipsan commented Nov 14, 2017

@DrMegavolt This is awesome 😄 🥇 Do you mind writing tests for the new code while at it? Also is package-lock.json needed?

If you're pressed on time I can join in and write tests tomorrow 🙂

@DrMegavolt
Copy link
Contributor Author

@stipsan
tests added.
rm package-lock

I can add hscan sscan in the next PR using new scan-helper.

@stipsan stipsan merged commit aae16cd into stipsan:master Nov 15, 2017
@stipsan
Copy link
Owner

stipsan commented Nov 15, 2017

Cool, merged it in 👍 I’ll slice a release after I’ve gone over the other PRs and updated the changelog 😃

stipsan added a commit that referenced this pull request Nov 15, 2017
## New Commands

### SCAN command (#334 @DrMegavolt)

```js
redis
  .scan(0, 'MATCH', 'foo*', 'COUNT', 1)
  .then(result => console.log(result))
```
 
### LRANGE command (#335 @sseidametov)

```js
redis
  .lrange('myKey', 0, 2)
  .then(result => console.log(result))
```
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.

2 participants