Updating templates for recent binding changes #33
Conversation
| @@ -1,5 +1,5 @@ | |||
| module.exports = function (context, req) { | |||
| context.log('Node.js HTTP trigger function processed a request. Name=' + req.originalUrl); | |||
| context.log('Node.js HTTP trigger function processed a request. RequestUri=' + req.originalUrl); | |||
|
|
But be aware that it goes live as soon as you merge. So if it's dependent on your script changes, let's not merge till those are live.
| @@ -7,7 +7,7 @@ | |||
| }, | |||
| { | |||
| "type": "table", | |||
There was a problem hiding this comment.
All of the json files have an extra level of unecessary indenting. Anyone know why, or whose editor is doing this?
There was a problem hiding this comment.
I believe @soninaren is using VS Code with the default indentation configuration (4 spaces) and that has updated our JSON files.
There was a problem hiding this comment.
So what is everyone else using, 4 spaces? Both my machines are using 2 (VS Enterprise) and I don't recall changing any defaults.
There was a problem hiding this comment.
VS uses 4 spaces for me for JSON. Not sure what;s the default, but we should all use the same...
|
I'll hold off committing this until we update, since as David says these updates rely on runtime updates. |
|
this has been merged in already. |
Absorbing changes from Azure/azure-functions-host@81ee8b7
Also making a few additional cleanups/fixes. E.g., I'm adding some missing samples to the sln file. I added the sln as a way for us to have a project to open to facilitate working on samples. I realize some people use Sublime, but I don't, so keeping this in sync helps me alot. What are others using? Either we keep the sln in sync or remove it - I'm for keeping it :)