-
Notifications
You must be signed in to change notification settings - Fork 3.9k
Few typo fixes #1827
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
Few typo fixes #1827
Conversation
sazzad16
left a comment
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.
@gkorland Great catches! But similar typos should also be found in BinaryJedis and may be some other classes. Could you please look for those and fix? Thanks!
|
more more typo fixes |
sazzad16
left a comment
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.
L1292: Change have to leave.
sazzad16
left a comment
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.
LGTM!
@marcosnils @HeartSaVioR As these are minor changes, I'm setting it to READY TO MERGE directly.
* Few typo fixes * Fix more typos * replace leave with have
* Few typo fixes * Fix more typos * replace leave with have
* Few typo fixes * Fix more typos * replace leave with have
No description provided.