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

New rule: no-async-without-await #81

Closed
macklinu opened this issue Feb 16, 2018 · 3 comments
Closed

New rule: no-async-without-await #81

macklinu opened this issue Feb 16, 2018 · 3 comments
Labels

Comments

@macklinu
Copy link
Collaborator

A follow up to the discussion in #78 (comment), it could be nice to have a rule like Ava's no-async-fn-without-await.

@alexilyaev
Copy link

Is this what you mean?
https://eslint.org/docs/rules/require-await

@SimenB
Copy link
Member

SimenB commented Apr 9, 2018

Heh, yep! There's a difference between test code and application code though - in tests it's always safe to remove, but in application code, the API is changed (the function no longer implicitly returns a promise). But we might not need it here then, thoughts?

@SimenB SimenB added the new rule label Apr 9, 2018
@SimenB
Copy link
Member

SimenB commented Jul 23, 2019

People can use overrides to enable/disable the core rule for any files they want

@SimenB SimenB closed this as completed Jul 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants