-
Notifications
You must be signed in to change notification settings - Fork 42
adds sam.yaml for publishing to SAR #17
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,18 @@ | ||
AWSTemplateFormatVersion: '2010-09-09' | ||
Transform: AWS::Serverless-2016-10-31 | ||
Description: > | ||
deno-runtime | ||
|
||
Example custom runtime for Deno | ||
|
||
Resources: | ||
DenoRuntime: | ||
Type: AWS::Lambda::LayerVersion | ||
Properties: | ||
Content: | ||
S3Bucket: begin-public-east-1 | ||
S3Key: deno-runtime.zip | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is this deno-lambda-layer.zip ? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It may be the s3 thing is not required (and can use ContentUri): https://serverless.pub/sar-layers/ There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
yes
maybe! seems the docs are pretty thin and maybe in flux; typical aws! =P There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Didn't work without s3! AWS docs seem very thin :) |
||
|
||
Outputs: | ||
DenoRuntimeArn: | ||
Value: !Ref DenoRuntime |
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.
Do you think this would make sense to be called simply "Deno" ? or is it important it's suffixed with Runtime?
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.
doesn't really matter / just a clarity thing / end user probably won't see it
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.
In serverless it's very fussy about being called DenoLambdaLayer.