chore(ci): update layer script to include event-handler#4956
chore(ci): update layer script to include event-handler#4956dreamorosi merged 3 commits intomainfrom
Conversation
dreamorosi
left a comment
There was a problem hiding this comment.
Thanks for the PR.
It's not a one-liner, or I'd have done it right away.
We also need to come up with some kind of test case to be added here to make sure it doesn't break the layer.
|
Ah I see. And it's a sort of chicken and egg situation cos you can't test it until the code is in the layer. |
No, the test builds the layer with whatever is in the build script you modified before running - that's the point of that canary. |
|
|
@sdangol - I had to make a commit to fix the PR. The end to end tests are passing, so this is ready to review and merge. |



Summary
The event-handler package is not included in the script that builds our lambda layers so we need to add it.
Changes
layers/src/layer-publisher-stack.tsscript to include the event-handler packageIssue number: closes #4955
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.