Skip to content
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.

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

Add unawaited_futures to lints #86

Closed
Alvarenga1 opened this issue Oct 19, 2022 · 1 comment
Closed

Add unawaited_futures to lints #86

Alvarenga1 opened this issue Oct 19, 2022 · 1 comment

Comments

@Alvarenga1
Copy link

Is there any reason why the rule unawaited_futures was removed ?

From what I understand not awating for a future is considered an error, and we should use unawaited for such cases.
Not waiting for a future to complete can cause all sorts of issues and sometimes it's not immediately obvious that a future is not being awaited, I think it would be a good idea to have this rule added back to the lints.

@devoncarew
Copy link
Member

Ah, there was discussion about unawaited_futures here: #25 (closing this issues as a dup).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants