-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Serverless Code Version - required #66
Comments
Yep, that sounds about right. First reflex: don't use SAM :P Second reflex: easy fix. Do you need it badly? Also, isn't it about bed time for you? |
Can we pull in the latest version of the SAM spec from here |
I think this is the latest one https://github.com/awslabs/goformation/blob/master/generate/sam-2016-10-31.json and I think it's aligned to the one we have checked-in. |
But we get the model from a project called |
We'll have to deal with this on a case-by-case basis. |
Howdy!
According to the serverless docs : https://github.com/awslabs/serverless-application-model/blob/master/versions/2016-10-31.md#s3-location-object
The version of a code thing isn't required
But we've got it marked non optional in our code
( i think it's sourced from : https://github.com/awslabs/aws-cdk/blob/4a7eaa6846d54c383022266809e3c3d3100992b6/packages/aws-cdk-resources/cloudformation-specs/000_sam.spec.json#L197 )
And putting a dummy value (like
0
is super not valid)The text was updated successfully, but these errors were encountered: