Skip to content

Conversation

@ada4a
Copy link
Contributor

@ada4a ada4a commented Oct 21, 2025

changelog: [unnecessary_find_map]: make diagnostic spans more precise
changelog: [unnecessary_filter_map]: make diagnostic spans more precise

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Oct 21, 2025
@rustbot
Copy link
Collaborator

rustbot commented Oct 21, 2025

r? @dswij

rustbot has assigned @dswij.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@ada4a ada4a force-pushed the unnecessary_filter_map branch from bedcc8d to 3fa518c Compare October 21, 2025 16:42
@ada4a ada4a changed the title unnecessary_filter_map: make diagnostic spans more precise unnecessary_{find,filter}_map: make diagnostic spans more precise Oct 21, 2025
@ada4a ada4a force-pushed the unnecessary_filter_map branch 2 times, most recently from aad18c8 to 018e0e2 Compare October 21, 2025 16:55
@ada4a ada4a force-pushed the unnecessary_filter_map branch from 018e0e2 to 0410ad2 Compare October 21, 2025 22:25
@rustbot
Copy link
Collaborator

rustbot commented Oct 21, 2025

This PR was rebased onto a different master commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

Copy link
Member

@dswij dswij left a comment

Choose a reason for hiding this comment

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

@dswij dswij added this pull request to the merge queue Oct 23, 2025
@dswij dswij removed this pull request from the merge queue due to a manual request Oct 23, 2025
@dswij
Copy link
Member

dswij commented Oct 23, 2025

Actually, @ada4a can you help to squash the commits?

@ada4a
Copy link
Contributor Author

ada4a commented Oct 23, 2025

I'd prefer to keep the last commit separate, as it's the one that contains the actual changes, while the first two are a general clean-up and thus could be squashed together. Would that be okay for you?

ada4a added 2 commits October 23, 2025 19:02
- introduce `Kind`: a bit more type-safe and hopefully a bit faster
@ada4a ada4a force-pushed the unnecessary_filter_map branch from 0410ad2 to 9145cee Compare October 23, 2025 17:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants