Closed jakub-qg closed 8 months ago
Hi @jakub-qg 👋🏻
Thanks a lot for your contribution 🙇🏻 Much appreciated 👏🏻
I'm taking a closer look now, but I wanted to give you a heads-up that I took the liberty to add a couple of commits to your PR, fixing some tests that went red with your changes. Lambda creates an S3 bucket in localstack implicitly, and that broke some of the S3 tests.
We have recently changed the ownership of the library, and that broke the CI running the tests automatically, so it appears normal that we both missed that. I'm in the process of fixing it for future PRs 👍
Awesome, thank you!
this is a very rough impl. of Lambda Invoke, probably good/stable enough for further development given this is an alpha lib according to semver