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

Feature Request: Async/Await #56

Open
Exitare opened this issue Apr 23, 2020 · 5 comments
Open

Feature Request: Async/Await #56

Exitare opened this issue Apr 23, 2020 · 5 comments

Comments

@Exitare
Copy link
Contributor

Exitare commented Apr 23, 2020

Right now, the package lacks support of async await. Should this be implemented?

@jb3
Copy link
Owner

jb3 commented Apr 23, 2020

What is the back compatibility like for this? I like the premise of async/await syntax but I don't fully know the benefits over the promise system.

@Exitare
Copy link
Contributor Author

Exitare commented Apr 23, 2020

Async / Await is just syntactic sugar. There is no real change in functionality. You can write cleaner code in terms of avoiding callbacks.

@jb3
Copy link
Owner

jb3 commented Apr 23, 2020

I see. How far back is async/await supported. I assume we'd have to drop support for older node versions.

@Exitare
Copy link
Contributor Author

Exitare commented Apr 23, 2020

Earliest is Node v7.6
according to this article.
https://www.infoq.com/news/2017/02/node-76-async-await/

But v8 supports it for sure!

@jb3
Copy link
Owner

jb3 commented Apr 23, 2020

Okay this seems reasonable to me, this is a feature which I'm interested in having.

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