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.

Sign up for GitHub

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

Jump to bottom

Rename vertical to cloud, add failed execution check #4

Merged

nima-ap-zz merged 2 commits into master from csharp/execute-function

Jun 26, 2019

Contributor

nima-ap-zz commented Jun 26, 2019

All instances of vertical have been renamed to cloud
We did not perform a check on the status code of the response from the azure function's execution. Which means that unless the target function threw, we would not be able to detect failure such as 404 where the function is just not found in the local run-time. Added a check for this.

nima-ap added 2 commits

June 26, 2019 13:54


          Add check for not successful HTTP call to local Azure Function

3737e5f


          Change all instances of vertical to cloud

29f7fc5

nima-ap-zz requested a review from MGudgin

June 26, 2019 21:05

nima-ap-zz self-assigned this

MGudgin approved these changes

View reviewed changes

nima-ap-zz merged commit ca487c4 into master

nima-ap-zz deleted the csharp/execute-function branch

June 26, 2019 21:10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet