Skip to content
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 new data structure called Times #1696

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

eitanschreiber123
Copy link

add a new data structure called
times(times, callback, seed = 1) that calls callback with seed passed as the argument and passes the result to callback the times number of times and returns a Seq.Indexed of the results
For the type-definitions i wrote for flow but i still have to figure out for typescript for defining function types i tried Function, and ((_: any) => any) but both returned errors and the only other option is to use interfaces or the type keyword

@conartist6
Copy link
Contributor

See my comments on #1695. I think this idea makes more sense, but personally I'd still be inclined not to do it. It's just more difficult to read than Range().map(_ => ...).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants