-
Notifications
You must be signed in to change notification settings - Fork 56
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 empathy fixes #441
Conversation
…will default to 200
@@ -5,7 +5,7 @@ const { | |||
} = require('../lib/commonOpts'); | |||
const list = require('./functions/list'); | |||
const deploy = require('./functions/deploy'); | |||
const test = require('./functions/test'); | |||
const server = require('./functions/server'); |
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.
+1 for this change; it's much more intuitive.
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.
@miketalley, these changes make sense to me. I think that they make the experience spinning up a server much more developer-friendly--I appreciate that we've added more logging when installing dependencies and that we've streamlined our error handling.
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.
Changes look good. Also tested and seems to work as expected.
Description and Context
This fixes several issues that were found during the empathy session on 2/3
Local Serverless Testing
hs functions test
tohs functions server
@abelbnpm install
process will be logged and interrupt the process @brandenrodgersres
If astatusCode
is provided.env
file, but having specified secrets inserverless.json
Serverless Deploy
cdnUrl
is provided @bkrainer @gkemp94