add promise example

[talves]

With the new Node.js 8.10 runtime, there are new handler types that can be declared with the “async” keyword or can return a promise directly.

For completion, I propose to add the example to return a promise

Please make changes as you see fit. 😀

[talves]

I was hoping this project had deploy previews turned on. (hint, hint!)

[swyxio]

just turned it on.. hopefully that works

as far as this one is concerned - i dont want to encourage this because using async/await already implies returning a promise. you're using both here.

what if we took out the async keyword? thats possible. i believe thats what your original source is trying to say. however i still wouldnt want to encourage it. basically if you can use async/await then theres no scenario in which you prefer returning a Promise. you can still do it, of course.

[talves]

Yeah, I understand. I don't think you can remove the async without returning a promise directly without using the callback. That is why I thought for completeness we should add this example, so people understand, but it may be confusing. They are the same thing really, just two acceptable ways to create an async function in lambda.

[netlify]

Deploy success!

Built with commit 0abb893

https://deploy-preview-17--festive-nightingale-47f180.netlify.com

[swyxio]

alrighty we agree. if its alright with you im just gonna close without merging. thanks a lot for thinking it thru with me

[talves]

Sounds good, the example is there if anyone needs it.