-
Notifications
You must be signed in to change notification settings - Fork 582
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
feat(collections): Add withoutAll that returns an array excluding all given values #1100
feat(collections): Add withoutAll that returns an array excluding all given values #1100
Conversation
I can take a first look tonight :-) |
ad4eec1
to
1030b3a
Compare
Co-authored-by: Yoshiya Hinosawa <stibium121@gmail.com>
Co-authored-by: Leon Strauss <me@lionc.de>
Co-authored-by: Leon Strauss <me@lionc.de>
…lis-Labs/deno_std into feat-collection-without-all
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.
Just the example left, the redt looks good to me
Co-authored-by: Leon Strauss <me@lionc.de>
Co-authored-by: Zhangyuan Nie <yuan@znie.org>
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 now, pinging @kt3k for re-review
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.
@getspooky LGTM. Nice work! Thanks!
Thanks @LionC @zhangyuannie for reviewing!
Returns an array excluding all given values, For performance if given array is large size(bigger than
200
) , we store values inSet
datastructure.Example:
Discussion
Please see #1065