Skip to content

fix: unnecessary_lazy_evaluations suggests wrongly for async closure #14644

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Apr 17, 2025
Merged

fix: unnecessary_lazy_evaluations suggests wrongly for async closure #14644

merged 1 commit into from
Apr 17, 2025

Conversation

profetia
Copy link
Contributor

@profetia profetia commented Apr 17, 2025 •
edited by rustbot
Loading

Closes #14578

changelog: [unnecessary_lazy_evaluations] fix wrong suggestions for async closure

@rustbot
Copy link
Collaborator

rustbot commented Apr 17, 2025

r? @llogiq

rustbot has assigned @llogiq.
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

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Apr 17, 2025
@profetia profetia marked this pull request as ready for review April 17, 2025 13:19
@samueltardieu
Copy link
Contributor

r? @samueltardieu

@rustbot
Copy link
Collaborator

rustbot commented Apr 17, 2025

samueltardieu is on vacation.

Please choose another assignee.

@samueltardieu
Copy link
Contributor

Funny, I thought I could still assign myself when in vacation. Well.

@samueltardieu samueltardieu added this pull request to the merge queue Apr 17, 2025
Merged via the queue into rust-lang:master with commit a49ea2d Apr 17, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@samueltardieu samueltardieu samueltardieu approved these changes

Assignees

@llogiq llogiq

Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

unnecessary_lazy_evaluations is suggested for async closure
4 participants
@profetia @rustbot @samueltardieu @llogiq