add, test, and document `includes`

RobinMalfait / lazy-collections

Collection of fast and lazy operations

MIT License

409 stars 24 forks source link

add, test, and document `includes` #24

Closed AlexVipond closed 1 year ago

AlexVipond commented 1 year ago

Matches functionality of Array.prototype.includes, except that it doesn't support a negative fromIndex, to avoid exhausting the full iterator. Followed the lead of slice when making that decision.

AlexVipond commented 1 year ago

@RobinMalfait thoughts on this PR and #25?

RobinMalfait commented 1 year ago

Hey, thanks so much for this contribution! 🙏

I rebased the branch and made some adjustments in #29, but made sure to still include you as a contributor. I also made a recent change such that an insiders release is published so that you can already play with it:

npm install lazy-collections@insiders