feat: Add ability to package LLRT as a layer#20
Conversation
|
Hi @tmokmss, I didn't change the package.json so thought it was best to avoid committing any updates to the package-lock.json files but it seems that won't pass the build so I've added the changes for these now 🤞 |
|
Hi @tmokmss , I'm just testing all the changes above and came across a strange bug. I think I've got to the bottom of it, but wanted to get your thoughts. Where you define the Would it be better to compare the name fields: const arch = props.architecture?.name == Architecture.ARM_64.name ? 'arm64' : 'x64';I don't imagine this would happen often, but I think it is safer. Wdyt? |
|
Hey @tmokmss , I've pushed all my changes. I've reverted the |
tmokmss
left a comment
There was a problem hiding this comment.
Hi I fixed the build issue and confirmed it works in my environment. Thanks!
|
Thanks! |
New feature to package LLRT in a layer so that it can be shared across multiple lambdas with the same binary type.
closes #19