-
Notifications
You must be signed in to change notification settings - Fork 374
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
Support for forEachRight
#107
Comments
Sure! I think we might just implement it for arrays. |
I've implemented a forEachRight function: export function forEachRight<T>(arr: T[] = [], callbackfn: (value: T, index: number, arr: T[]) => void): void {
for (let i = arr.length - 1; i >= 0; i--) {
const element = arr[i];
callbackfn(element, i, arr);
}
} If this implementation looks good, may I proceed with enhancing the documentation and tests, and then submit a pull request? |
Looks great! What about renaming |
Oh, I think the code you suggested is better too. |
Hello,
If you need loadsh's forEachRight function, Can I work it?
If it is okay to proceed, lodash's forEachRight supports both Array and Object, but in [#91], I would like to ask if it is only necessary to implement array in the array category.
The text was updated successfully, but these errors were encountered: