Skip to content
This repository has been archived by the owner on Feb 2, 2021. It is now read-only.

Update user guide to add new DNS and remove docker requirement #188

Merged
merged 1 commit into from
Feb 11, 2020

Conversation

Waterdrips
Copy link
Contributor

Description

We added cloudflare DNS as an option, but it's not in the guide.
We also removed the need for Docker by creating the registry-login
command.

Signed-off-by: Alistair Hey [email protected]

How Has This Been Tested?

visually

Checklist:

I have:

  • checked my changes follow the style of the existing code / OpenFaaS repos
  • updated the documentation and/or roadmap in README.md
  • read the CONTRIBUTION guide
  • signed-off my commits with git commit -s
  • added unit tests

We added cloudflare DNS as an option, but it's not in the guide.
We also removed the need for Docker by creating the registry-login
command.

Signed-off-by: Alistair Hey <[email protected]>
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 85efcab into openfaas:master Feb 11, 2020
@Waterdrips Waterdrips deleted the update_user_guide branch June 2, 2020 10:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants