rust-itertools / itertools

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

Remove deprecated #878

Closed Philippe-Cholet closed 7 months ago

Philippe-Cholet commented 7 months ago

At the same time as #877, I listed our deprecated parts:

Deprecated Alternative Comment
Itertools::group_by
since 0.13.0
Itertools::chunk_by Newly deprecated.
Do not remove!
unfold
since 0.13.0
core::iter::from_fn
MSRV ≥ 1.34.0
Newly deprecated.
Do not remove!
Itertools::fold1
since 0.10.2
Iterator::reduce
MSRV ≥ 1.51.0
Unavailable alternative
since MSRV = 1.43.1
Do not remove!
zip
since 0.10.4
core::iter::zip
MSRV ≥ 1.59.0
Unavailable alternative
since MSRV = 1.43.1
Do not remove!
repeat_call
since 0.8.0
core::iter::repeat_with
MSRV ≥ 1.28.0
Remove?!
Itertools::foreach
since 0.8.0
Iterator::for_each
MSRV ≥ 1.21.0
Remove?!
Itertools::step
since 0.8.0
Iterator::step_by
MSRV ≥ 1.28.0
Remove?!
Itertools::map_results
since 0.10.0
Itertools::map_ok Remove?!
Itertools::fold_results
since 0.10.0
Itertools::fold_ok Remove?!

Should we remove repeat_call, Itertools::{foreach, step, map_results, fold_results} (which is what I did here)? Or when should we?

Note: 0.8.0 and 0.10.0 are resp. 5 and 3 years old.

codecov[bot] commented 7 months ago

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (6814180) 94.38% compared to head (1dc7bce) 94.53%. Report is 10 commits behind head on master.

Files Patch % Lines
src/sources.rs 50.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #878 +/- ## ========================================== + Coverage 94.38% 94.53% +0.15% ========================================== Files 48 48 Lines 6665 6574 -91 ========================================== - Hits 6291 6215 -76 + Misses 374 359 -15 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Philippe-Cholet commented 7 months ago

The list of deprecations is not complete. I used some command line to make the list but it clearly did not do what I expected.

EDIT: It was only missing zip, list updated.

At first, I did not understand why semver-checks passed CI but it's simply because the version was bumped in Cargo.toml to 0.13.0. 🙄