feat(parallel): clamp the limit between 1 and array length

[MarlonPassos-git]

Tip

The owner of this PR can publish a preview release by commenting /publish in this PR. Afterwards, anyone can try it out by running pnpm add radashi@pr<PR_NUMBER>.

Summary

now:

const array = [1, 2, 3]
const doSomething = async (n: number) => { ...}
_.parallel(-1, array, doSomething) // run 1 function in paraller (in the current implementation no function is executed)
_.parallel(0, array, doSomething) // run 1 function in paraller (in the current implementation no function is executed)
_.parallel(100, array, doSomething) // run 3 function in paraller 
_.parallel(Infinity, array, doSomething) // run 3 function in paraller (in the current implementation this throws an error)

Related issue, if any:

https://github.com/orgs/radashi-org/discussions/235
sodiray/radash#426

For any code change,

• Related documentation has been updated, if needed
• Related tests have been added or updated, if needed
• Related benchmarks have been added or updated, if needed

Does this PR introduce a breaking change?

No

Bundle impact

Status	File	Size 1	Difference
M	src/async/parallel.ts	1698	+135 (+9%)

Footnotes

1. Function size includes the import dependencies of the function. ↩

[aleclarson]

Hey! There's a new requirement for PRs that introduce new features. Without this requirement met, we won't be able to merge this. Note that this PR can still be included in a beta prerelease before this requirement is fulfilled.

• Edit the next-minor.md file

⚠️ Note: You need to run git rebase main before this file will appear locally.

[aleclarson]

Postponing this until v12.4.0, since it conflicts with PR #262 and we don't have an easy way to solve that currently.

Basically, the prerelease tag triggers a GitHub CI workflow that attempts to cherry-pick the PR into the beta branch. If that fails, there's no way to manually fix the merge conflict. In the future, I could add a fail-safe to the CI workflow that would open a “clone” PR targeting the beta branch specifically. Then I could checkout that PR, manually fix the conflict, push the fix, and merge that PR into the beta branch, triggering a prerelease.

[MarlonPassos-git]

Postponing this until v12.4.0, since it conflicts with PR #262 and we don't have an easy way to solve that currently.

Basically, the prerelease tag triggers a GitHub CI workflow that attempts to cherry-pick the PR into the beta branch. If that fails, there's no way to manually fix the merge conflict. In the future, I could add a fail-safe to the CI workflow that would open a “clone” PR targeting the beta branch specifically. Then I could checkout that PR, manually fix the conflict, push the fix, and merge that PR into the beta branch, triggering a prerelease.

fixed 🫡

[radashi-bot]

A stable release 12.3.0 has been published to NPM. 🚀

To install:

pnpm add [email protected]