Skip to content
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

Add optional response status #9

Merged

Conversation

koffeinfrei
Copy link
Contributor

Adds an optional third status code as part of the return value.

Is based on #6 (can't change the base branch to a forked repo branch).

The headers weren't actually set previously.
The statement `headers = res_headers` doesn't have any effect, since it
only creates a local variable.

Signed-off-by: Alexis Reigel <mail@koffeinfrei.org>
@derek
Copy link

derek bot commented Oct 13, 2019

Thank you for your contribution. I've just checked and your commit doesn't appear to be signed-off. That's something we need before your Pull Request can be merged. Please see our contributing guide.
Tip: if you only have one commit so far then run: git commit --amend --signoff and then git push --force.

@derek derek bot added the no-dco label Oct 13, 2019
@koffeinfrei koffeinfrei force-pushed the feat/add-optional-response-status branch from d9449cd to 8834197 Compare October 13, 2019 16:14
@derek derek bot removed the no-dco label Oct 13, 2019
Signed-off-by: Alexis Reigel <mail@koffeinfrei.org>
@koffeinfrei koffeinfrei force-pushed the feat/add-optional-response-status branch from 8834197 to 5d8eae0 Compare October 13, 2019 16:20
Copy link
Member

@alexellis alexellis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 💪

@alexellis alexellis merged commit 5d193e6 into openfaas:master Oct 13, 2019
@alexellis
Copy link
Member

Merged.

@koffeinfrei koffeinfrei deleted the feat/add-optional-response-status branch October 13, 2019 19:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants