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

docs: update the sample #77

Merged
merged 1 commit into from
Jun 29, 2022
Merged

docs: update the sample #77

merged 1 commit into from
Jun 29, 2022

Conversation

mrwormhole
Copy link
Contributor

@mrwormhole mrwormhole commented Jun 12, 2022

Signed-off-by: F. Talha Altinel [email protected]

Description

did an update on the docs, removed ioutil usage

How Has This Been Tested?

How are existing users impacted? What migration steps/scripts do we need?

N/A

Checklist:

I have:

  • updated the documentation and/or roadmap (if required)
  • read the CONTRIBUTION guide
  • signed-off my commits with git commit -s
  • added unit tests

Signed-off-by: F. Talha Altinel <[email protected]>
@mrwormhole mrwormhole changed the title updated the docs docs: update the sample Jun 12, 2022
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 7fccf7b into openfaas:master Jun 29, 2022
@mrwormhole
Copy link
Contributor Author

@alexellis I did a similar one but for a different repository, just to keep you informed openfaas/templates#295

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