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
Promote AWS 2 Lambda to native #1730
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this javascript should be in the extension.
2c80454
to
a877d9a
Compare
so, remove the whole lib folder with the .js files? |
I see, the other extensions do not have it either, will remove the lib folder |
a877d9a
to
d6d9da7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @Jeansen ! A minor comment inline. Besides, if you could please use more descriptive commit messages next time. Issue title would be nice in addition to the issue number.
d6d9da7
to
7d5837e
Compare
It is perfect from my PoV now, thanks @Jeansen ! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot!
No description provided.