-
Notifications
You must be signed in to change notification settings - Fork 450
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 LPUSH & RPUSH command API wrappers to GarnetClient #382
Conversation
Is the file-scope namespace the matter here?!! |
/home/runner/work/garnet/garnet/libs/client/GarnetClientAPI/GarnetClientListCommands.cs(44,1): error WHITESPACE: Fix whitespace formatting. Replace 17 characters with '\n\s\s\s\s\s\s\s\s'. [/home/runner/work/garnet/garnet/libs/client/Garnet.client.csproj] |
Why you code style don't complain during the build in VS? |
@badrishc is the process takes long to review, because I need to update to main whenever a new PR is merged |
We will review it in the next day. Also note:
|
@microsoft-github-policy-service agree |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your contribution! Added some comments...
Seems the review takes so long :) |
/cc badrishc