-
Notifications
You must be signed in to change notification settings - Fork 2.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
upgrade to nodejs12.x for all examples and tests #1363
Conversation
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.
Haha that hit a lot of files. Do you know of anything that could cause the examples to not work after bumping the node version?
@@ -9,7 +9,7 @@ LambdaEdgeFunctionSample: | |||
Type: AWS::Serverless::Function | |||
Properties: | |||
CodeUri: src/ | |||
Runtime: nodejs6.10 | |||
Runtime: nodejs12.x |
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.
Following the lambda@edge requirements guide node12 isn't supported for lambda@edge!
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.
@timoschilling thank you for checking this. I will update the PR.
e6a587e
to
8cc084f
Compare
8cc084f
to
a8a2ba1
Compare
Codecov Report
@@ Coverage Diff @@
## develop #1363 +/- ##
========================================
Coverage 94.42% 94.42%
========================================
Files 78 78
Lines 4555 4555
Branches 912 912
========================================
Hits 4301 4301
Misses 119 119
Partials 135 135 Continue to review full report at Codecov.
|
Issue #, if available:
Description of changes:
updated nodejs version in tests and examples to nodejs12.x
Description of how you validated changes:
Checklist:
make pr
passesexamples/2016-10-31
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.