-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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 mode() function & tests #7199
Conversation
@jwilder can I also get a thumbs up that this is OK to merge into 1.0 to be released with the next release candidate? |
@jsternberg I think i signed the CLA with this account (@agaurav), so please use this. |
Looking at the CLA responses we've gotten, I see you used @agaurav as the GitHub account name, but you used Either one is fine with me. |
Yeah you can use that. |
Looks great @agaurav! This is missing a changelog entry though. |
cc @nathanielc |
@jwilder #7207 for changelog. |
No description provided.