-
Notifications
You must be signed in to change notification settings - Fork 655
feat(collections): add index arg to methods #6382
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
base: main
Are you sure you want to change the base?
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #6382 +/- ##
=======================================
Coverage 94.16% 94.17%
=======================================
Files 592 592
Lines 42975 42990 +15
Branches 6807 6807
=======================================
+ Hits 40466 40484 +18
+ Misses 2458 2456 -2
+ Partials 51 50 -1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
I'm generally in favor of the idea. It looks useful in several places, and also this is aligned to the design of builtin map methods (such as What do people think? Note: Currently we have policy of accepting any new feature in |
I like it. |
I'm in favour. |
@kt3k should we add this in? |
I'm more than happy to rebase this if we want to move forward with it. Just let me know! |
I suggest rebasing anyway 🙂 |
bff3bbf
to
c0c394e
Compare
Adds an
index
arg to collections' predicate/selector/transformer/discriminator method. I added it only to collection helpers that operate on iterables because collection helpers that operate on records don't seem as likely to need access to the key index. Please let me know if I missed any.Closes #6288