-
-
Notifications
You must be signed in to change notification settings - Fork 485
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: add array/base/cunone-by
package
#2835
Conversation
This commit contains all inital changes for adding the `base/cunone-by` package. Ref: stdlib-js#2325 Co-authored-by: Mohammad Kaif <mdkaifprofession@gmail.com>
This commit aims to add the `base/cunone-by` package. Ref: stdlib-js#2325 Co-authored-by: Mohammad Kaif <mdkaifprofession@gmail.com>
@stdlib/array/base/cunone-by
package@stdlib/array/base/cunone-by
package
@stdlib/array/base/cunone-by
package@stdlib/array/base/cunone-by
package
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!
Left a few comments concerning some doc fixes, but overall this looks very good!
PR-URL: stdlib-js#2835 Co-authored-by: Mohammad Kaif <mdkaifprofession@gmail.com>
Hey @Planeshifter I have made the necessary changes. Is there anything else that I need to add |
Signed-off-by: Philipp Burckhardt <pburckhardt@outlook.com>
@stdlib/array/base/cunone-by
packagearray/base/cunone-by
package
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.
Let's land this! Thanks again for the PR!
Resolves #2325
Description
This pull request:
@stdlib/array/base/cunone-by
package that cumulatively tests whether no array element in a provided array passes a test implemented by a predicate functionRelated Issues
This pull request:
@stdlib/array/base/cunone-by
#2325Questions
No.
Other
No.
Checklist
@stdlib-js/reviewers