Skip to content

Conversation

@aatifsyed
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Nov 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.75%. Comparing base (6814180) to head (a03c199).
⚠️ Report is 169 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1073      +/-   ##
==========================================
- Coverage   94.38%   93.75%   -0.64%     
==========================================
  Files          48       50       +2     
  Lines        6665     6326     -339     
==========================================
- Hits         6291     5931     -360     
- Misses        374      395      +21     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@phimuemue
Copy link
Member

phimuemue commented Nov 4, 2025

Hashmap is only available under use_std (see rust-lang/rust#56192 for why alloc is not sufficient). Can you fix this?

peeking_next_by_clone! { ['a, T] alloc::collections::linked_list::Iter<'a, T> }
#[cfg(feature = "use_alloc")]
peeking_next_by_clone! { ['a, T] alloc::collections::vec_deque::Iter<'a, T> }
const _: () = {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please replace then const _ with repeated gates.

If we start adopting this pattern, we should do it consistently and not arbitrarily only here.

phimuemue

This comment was marked as outdated.

Copy link
Member

@phimuemue phimuemue left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you fix CI and squash the commits?

@phimuemue phimuemue force-pushed the aatifsyed/peeking-next-btree branch from 8c7e761 to 556ed1a Compare December 2, 2025 13:04
@phimuemue
Copy link
Member

Hi @aatifsyed, I'm unsure. If you want to fix these, I'd happily merge it.

@aatifsyed
Copy link
Contributor Author

The semver checks failure seems spurious.

@aatifsyed aatifsyed force-pushed the aatifsyed/peeking-next-btree branch from df58323 to a03c199 Compare December 2, 2025 14:00
@aatifsyed
Copy link
Contributor Author

squashed :)

Copy link
Member

@phimuemue phimuemue left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, nicely done.

@phimuemue phimuemue added this pull request to the merge queue Dec 2, 2025
Merged via the queue into rust-itertools:master with commit 3f99dc6 Dec 2, 2025
12 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants