-
-
Notifications
You must be signed in to change notification settings - Fork 481
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/cuany-by-right
#2667
Conversation
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.
👋 Hi there! 👋
And thank you for opening your first pull request! We will review it shortly. 🏃 💨
lib/node_modules/@stdlib/array/base/cuany-by-right/package-lock.json
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/array/base/cuany-by-right/lib/assign.js
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/array/base/cuany-by-right/benchmark/benckmark.js
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/array/base/cuany-by-right/benchmark/benchmark.assign.length.js
Outdated
Show resolved
Hide resolved
@stdlib/array/base/cuany-by-right
(Cuany by right) array/base/cuany-by-right
@DevMhrn Additionally, before this PR can be reviewed again and merged, all CI failures will need to be addressed. |
/stdlib lint-autofix |
@kgryte , I have updated some lines of code , by correcting some things , could you please review it |
@kgryte please review it and let know the possible error or corrections, to contibute in this repo . |
lib/node_modules/@stdlib/array/base/cuany-by-right/benchmark/benchmark.assign.length.js
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/array/base/cuany-by-right/benchmark/benckmark.length.js
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/array/base/cuany-by-right/docs/repl.txt
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/array/base/cuany-by-right/docs/types/index.d.ts
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/array/base/cuany-by-right/docs/types/index.d.ts
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/array/base/cuany-by-right/docs/types/test.ts
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/array/base/cuany-by-right/examples/index.js
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/array/base/cuany-by-right/examples/index.js
Outdated
Show resolved
Hide resolved
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.
This PR is still in need of significant clean-up. Before any further review can happen, all lint errors need to be resolved. Please consult existing packages in array/base
and emulate the style and conventions as closely as you can and as appropriate.
array/base/cuany-by-right
array/base/cuany-by-right
PR Commit Message
Please review the above commit message and make any necessary adjustments. |
PR Commit Message
Please review the above commit message and make any necessary adjustments. |
Resolves #2327
Description
This pull request introduces the
@stdlib/array/base/cuany-by-right
package, which provides functions for iterating over an array from right-to-left to determine if any element satisfies a given predicate function. The package includes the following functions:cuanyByRight
, but allows setting output values in a provided output array with support for offset and stride.Purpose
The main goal of this pull request is to add the
@stdlib/array/base/cuany-by-right
package to the codebase.Related Issues
@stdlib/array/base/cuany-by-right
#2327@stdlib/array/base/cuany-by-right
#2327Questions for Reviewers
None.
Additional Information
None.
Checklist
@stdlib-js/reviewers