rust-itertools / itertools

Extra iterator adaptors, iterator methods, free functions, and macros.
https://docs.rs/itertools/
Apache License 2.0
2.72k stars 309 forks source link

Maintenance #413

Open jplatte opened 4 years ago

jplatte commented 4 years ago

Hi! It seems to me that itertools is quite low on maintenance power – a fairly big amount of PRs have the waiting-on-review tag, some are untagged and some issues also haven't received any comments.

Is there anything I can do to help?

Veetaha commented 4 years ago

@bluss, @phimuemue, @jswrenn could you clarify this?

jswrenn commented 4 years ago

When we receive a PR for a new method, we need to resolve:

If the contribution is an optimized specialization of an Iterator method (such as nth or fold), the specialization needs to be very thoroughly tested (preferably with quickcheck tests) to ensure it behaves identically (especially on edge cases, and with when it panics).

Reviews that resolve any of the above are very helpful!

The bar for inclusion in itertools is much lower than it is for the standard library, but we'd still like to know that additions will be helpful to more than just the person submitting it. If there's a PR proposing something you need, leave a note describing your use-case. This feedback is helpful for prioritizing PRs, too.

jplatte commented 4 years ago

Hm… I think I'm more concerned about things that aren't new features missing reviews (e.g. finally getting everything rustfmt'ed so I don't have to bypass my default format-on-save constantly). Of the 23 PRs that are currently open, only 8 are labeled 'waiting on review'. There's also lots of feature requests that have no answers from a maintainer. I usually ask whether a PR is welcome before writing the code. Should I not do that here? Should I just ping somebody if I don't get an answer after a while?