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

Add NULLSafe Commands #23

Merged
merged 3 commits into from Nov 24, 2015
Merged

Conversation

@russellpierce
Copy link
Contributor

@russellpierce russellpierce commented Nov 24, 2015

Adding NULLSafe wrappers to redisCommand and redisCommandArgv along with tests

…this treatment for the 'correct' handling of redis errors as R errors rather than text strings; this came up in testing under the condition of pinging a Redis server that had not been given the correct AUTH & changelog cleanup
@eddelbuettel
Copy link
Owner

@eddelbuettel eddelbuettel commented Nov 24, 2015

You're on fire :)

eddelbuettel added a commit that referenced this pull request Nov 24, 2015
Add NULLSafe Commands
@eddelbuettel eddelbuettel merged commit ab50827 into eddelbuettel:master Nov 24, 2015
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@Russell-DataScience Russell-DataScience deleted the russellpierce:feature/NULLSafe branch Nov 24, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.