Skip to content
This repository has been archived by the owner on Jan 11, 2023. It is now read-only.

Remove async/await #62

Closed
Rich-Harris opened this issue Jan 4, 2018 · 4 comments
Closed

Remove async/await #62

Rich-Harris opened this issue Jan 4, 2018 · 4 comments

Comments

@Rich-Harris
Copy link
Member

Currently, Sapper requires Node 8, which means you can't run it on AWS Lambda and what-have-you

@EmilTholin
Copy link
Member

Would you like to remove async/await from the tests as well?

@Rich-Harris
Copy link
Member Author

I suppose we'd need to, if we're to guard against regressions. If we're not testing on Node 6 it'd be very easy to accidentally stop supporting it

@thgh
Copy link
Contributor

thgh commented Jan 20, 2018

I would vote to use @std/esm instead. Not sure what the performance impact would be on Lambda.

@Rich-Harris
Copy link
Member Author

@thgh @std/esm is just about supporting import and export, it doesn't have any bearing on whether async and await can be used. This issue should be closed though, it was fixed in #67

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

No branches or pull requests

3 participants