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
fix(AWS Layers): Support references to external layers #9826
Conversation
Codecov Report
@@ Coverage Diff @@
## master #9826 +/- ##
=======================================
Coverage 86.14% 86.14%
=======================================
Files 329 329
Lines 12808 12803 -5
=======================================
- Hits 11033 11029 -4
+ Misses 1775 1774 -1
Continue to review full report at Codecov.
|
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.
@pgrzesik looks good. I have just one suggestion
'LAMBDA_LAYER_REFERENCE_NOT_FOUND' | ||
); | ||
if (process.env.SLS_DEBUG) { | ||
serverless.cli.log( |
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.
It would be better to rely on @serverless/utils/log
We're rather on the path on removing dependency on serverless
instance.
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.
of course, great suggestion 👍
b0c53c2
to
ca0656f
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.
Looks great 👍
Closes: #9797
This fix will not error out if layer referenced by
Ref
will not be found locally, but rather allow such behavior and let the deployment fail is the layer really cannot be referenced.