it-gorillaz / aws-wave

AWS Lambda Microframework
MIT License
3 stars 0 forks source link

[question] Support for async/await #1

Open afgallo opened 6 years ago

afgallo commented 6 years ago

Hi there,

Thanks for this library, really like what you've done @tommelo 👍

I was wondering whether you've worked on a version that supports async/await? If not, do you think the changes would be around making execute and before async? https://github.com/it-gorillaz/aws-wave/blob/fa198c651837aa521e21fd7f0944ce1fffe286f8/lib/handler/abstract-request-handler.js#L76

Thanks, Andre

tommelo commented 6 years ago

Hey @afgallo, thanks, I appreciate that!

Actually, this lib needs a huge refactor, everything was done with a tight deadline at the company. I'll look forward to make some improvements(including async/await).

If you have any other suggestion or If you want to contribute with code, design, tests and etc, just let me know, that would be great!

afgallo commented 6 years ago

When you say huge refactor do you mean a complete re-write? If you could list out some items that are in your head we can maybe come up with a little roadmap of improvements 😉

tommelo commented 6 years ago

Yes, a complete re-write! I guess we could start trying to figure out a better design for the following features:

Then, when we get this done, it would be great to have a cli tool to create project "boilerplates" or "templates" (controllers, validators, cloudformation templates and etc).

Let me know what you think!