Skip to content

Conversation

zaidoon1
Copy link

to address #148 (comment)

@marvin-j97 marvin-j97 added enhancement New feature or request api labels Sep 27, 2025
@zaidoon1 zaidoon1 changed the title update drop_range API to use RangeBounds with inclusive bounds update drop_range API to use RangeBounds Sep 28, 2025
@zaidoon1 zaidoon1 force-pushed the zaidoon/DropRange branch 2 times, most recently from 884cdd0 to 677653e Compare September 28, 2025 16:32
@zaidoon1
Copy link
Author

zaidoon1 commented Sep 28, 2025

linter failure is related to:

error: the variable `count` is used as a loop counter
   --> src/tree/mod.rs:149:9
    |
149 |         for (key, value) in iter {
    |         ^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `for (count, (key, value)) in iter.enumerate()`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#explicit_counter_loop

    
but I did't touch this code?

@marvin-j97
Copy link
Contributor

linter failure is related to:

error: the variable `count` is used as a loop counter
   --> src/tree/mod.rs:149:9
    |
149 |         for (key, value) in iter {
    |         ^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using: `for (count, (key, value)) in iter.enumerate()`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#explicit_counter_loop

    
but I did't touch this code?

I think the linter rules changed again - I updated the 3.0.0 branch to allow it.

@marvin-j97 marvin-j97 merged commit 7084eee into fjall-rs:3.0.0 Sep 29, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants