Navigation Menu

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

Update docs to clarify registry-login command #221

Merged
merged 1 commit into from Sep 16, 2020

Conversation

Waterdrips
Copy link
Contributor

@Waterdrips Waterdrips commented Sep 12, 2020

Description

The registry-login usage command was copied from faas-cli and doent look
like it was changed, the password-stdin flag was referencing the gateway
password.

This also changes the docs to specify pressing ctrl+d when trying a
password to close the input.

Spotted these when looking into #220

Signed-off-by: Alistair Hey alistair@heyal.co.uk

How Has This Been Tested?

building and running the command on linux

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

The registry-login usage command was copied from faas-cli and doent look
like it was changed, the password-stdin flag was referencing the gateway
password.

This also changes the docs to specify pressing ctrl+d when trying a
password to close the input.

Signed-off-by: Alistair Hey <alistair@heyal.co.uk>
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.

Approved

@alexellis alexellis merged commit be63099 into openfaas:master Sep 16, 2020
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.

None yet

2 participants